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: remove cancel-in-progress from release.yml #876

Merged
merged 2 commits into from
Sep 22, 2023

Conversation

JoshuaKGoldberg
Copy link
Owner

PR Checklist

Overview

Switches actions/checkout@4 to fetch from ref: main (the branch) instead of the commit that triggered the workflow. Doing so should mean that every release workflow runs on the latest commit as of when it starts.

This should mean that if N release-worthy commits occur quickly:

  1. The first one should always finish releasing nicely
  2. The remaining 1...N commits will each

I don't love that this solution results in 1...N release workflows each deleting branch protection, doing nothing, then recreating the branch protection. But given https://github.com/orgs/community/discussions/12835 I don't see a way around it. The branch protection shenanigans should be made better by #145.

@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Merging #876 (55066c1) into main (b077fbc) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #876   +/-   ##
=======================================
  Coverage   92.82%   92.82%           
=======================================
  Files          89       89           
  Lines        4629     4629           
  Branches      355      355           
=======================================
  Hits         4297     4297           
- Misses        330      331    +1     
+ Partials        2        1    -1     
Flag Coverage Δ
create 70.69% <100.00%> (ø)
initialize 35.50% <50.00%> (ø)
migrate 70.19% <100.00%> (ø)
unit 59.71% <50.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
...s/writing/creation/dotGitHub/createWorkflowFile.ts 100.00% <100.00%> (ø)
src/steps/writing/creation/dotGitHub/workflows.ts 88.29% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

@JoshuaKGoldberg JoshuaKGoldberg merged commit f124518 into main Sep 22, 2023
16 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the no-cancel-in-progress branch September 22, 2023 07:18
@github-actions
Copy link

github-actions bot commented Oct 2, 2023

🎉 This is included in version v1.32.0 🎉

The release is available on:

Cheers! 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Bug: Releases can become broken after two quick pushes to main
1 participant