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

CI: Add concurrency to reduce runs for subsequent pushes to PRs #2893

Merged
merged 3 commits into from
Mar 17, 2023

Conversation

nilason
Copy link
Contributor

@nilason nilason commented Mar 17, 2023

Attempt to fix CI concurrency cancelling on main.

Inspired by #2868 (comment)

@nilason
Copy link
Contributor Author

nilason commented Mar 17, 2023

Cancelling on pull requests works as expected.
The code seems promising enough to work on main as well, I'll merge this as soon as possible following with a quick merge of #2873 for testing.

@nilason nilason added the CI Continuous integration label Mar 17, 2023
@nilason nilason merged commit 22c800d into OSGeo:main Mar 17, 2023
@nilason
Copy link
Contributor Author

nilason commented Mar 17, 2023

Looks like this works as expected. 22c800d and 29609fa were merged with very short interval, no cancellations.

@wenzeslaus
Copy link
Member

Looks good to me, too! Thanks!

ninsbl pushed a commit to ninsbl/grass that referenced this pull request Mar 20, 2023
Avoids cancelling of "pending jobs" on main. Previous solution only avoids
cancelling on "jobs in progress".
@wenzeslaus wenzeslaus changed the title CI concurrency CI: Add concurrency to reduce runs for subsequent pushes to PRs Jun 6, 2023
@neteler neteler added this to the 8.4.0 milestone Aug 16, 2023
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
Avoids cancelling of "pending jobs" on main. Previous solution only avoids
cancelling on "jobs in progress".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants