Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix release automation #1471

Merged
merged 7 commits into from
Oct 26, 2023
Merged

Fix release automation #1471

merged 7 commits into from
Oct 26, 2023

Conversation

t0yv0
Copy link
Member

@t0yv0 t0yv0 commented Oct 25, 2023

Editing the GitHub actions workflow to propagate new bridge releases to all providers.

Fixes #1470
Fixes #1474

@t0yv0
Copy link
Member Author

t0yv0 commented Oct 25, 2023

@codecov-commenter
Copy link

codecov-commenter commented Oct 25, 2023

Codecov Report

Merging #1471 (bd3879e) into master (167777d) will decrease coverage by 4.05%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1471      +/-   ##
==========================================
- Coverage   61.93%   57.88%   -4.05%     
==========================================
  Files         185      285     +100     
  Lines       32832    39602    +6770     
==========================================
+ Hits        20333    22924    +2591     
- Misses      11360    15341    +3981     
- Partials     1139     1337     +198     

see 104 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pulumi-bot pulumi-bot added the needs-triage Needs attention from the triage team label Oct 25, 2023
@t0yv0
Copy link
Member Author

t0yv0 commented Oct 25, 2023

Example PR: pulumi/pulumi-mailgun#208

@t0yv0 t0yv0 requested a review from iwahbe October 25, 2023 01:30
Copy link
Member

@iwahbe iwahbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine as it stands, but I would prefer that we pull the matrix from ci-mgmt. Ideally, we wouldn't maintain multiple lists of all our providers.

@t0yv0
Copy link
Member Author

t0yv0 commented Oct 25, 2023

You are right. Scripted drawing from ci-mgmt. Test-driving.

@t0yv0 t0yv0 merged commit 6a6731e into master Oct 26, 2023
73 checks passed
@t0yv0 t0yv0 deleted the t0yv0/fix-release-automation branch October 26, 2023 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Needs attention from the triage team
Projects
None yet
4 participants