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

Autocanceling previous tests when pushing changes #1938

Merged
merged 3 commits into from
Oct 12, 2022
Merged

Autocanceling previous tests when pushing changes #1938

merged 3 commits into from
Oct 12, 2022

Conversation

kafkasl
Copy link
Contributor

@kafkasl kafkasl commented Oct 12, 2022

The changes I added will cancel all the workflows (that share the same concurrency group) related to the current PR when new changes are pushed. I've added it only to the CI pipeline which is the most resource intensive one.

PR checklist

  • This comment contains a description of changes (with reason).
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@codecov
Copy link

codecov bot commented Oct 12, 2022

Codecov Report

Merging #1938 (cd2db47) into dev (6d7625a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1938   +/-   ##
=======================================
  Coverage   59.94%   59.94%           
=======================================
  Files          37       37           
  Lines        4664     4664           
=======================================
  Hits         2796     2796           
  Misses       1868     1868           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@matthdsm matthdsm left a comment

Choose a reason for hiding this comment

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

I like this! 👍🏻 (pending CI tests)

Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
@kafkasl kafkasl merged commit c84dfb7 into nf-core:dev Oct 12, 2022
This was referenced Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants