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: cancel in-progress jobs when new commit #1094

Merged
merged 4 commits into from
Feb 8, 2023
Merged

CI: cancel in-progress jobs when new commit #1094

merged 4 commits into from
Feb 8, 2023

Conversation

adamjstewart
Copy link
Collaborator

@adamjstewart adamjstewart commented Feb 7, 2023

See the docs.

If a new commit is pushed, there's no reason to finish jobs for old commits. RtD CI already does this.

The name of the concurrency group seems rather finicky. Other users recommend something else. Spack uses yet another option. It's unclear to me which is required. I suggest starting simple and adding complexity when needed.

@github-actions github-actions bot added the testing Continuous integration testing label Feb 7, 2023
@adamjstewart adamjstewart marked this pull request as ready for review February 7, 2023 22:26
@adamjstewart adamjstewart added this to the 0.4.1 milestone Feb 8, 2023
@adamjstewart adamjstewart merged commit d897e33 into main Feb 8, 2023
@adamjstewart adamjstewart deleted the ci/cancel branch February 8, 2023 14:38
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Continuous integration testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants