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

[skip circleci] Add option to effectively cancel prev runs #2310

Merged
merged 4 commits into from Oct 30, 2021

Conversation

jneira
Copy link
Member

@jneira jneira commented Oct 29, 2021

Without using skip-duplicate-actions which is failing due to github access permission. 
See fkirc/skip-duplicate-actions#103
@jneira jneira changed the title Add option to cancel prev runs Add option to effectively cancel prev runs Oct 29, 2021
@lance6716
Copy link

https://github.com/pingcap/dm/pull/1935/files

this is our repo's modification, for your reference

@jneira
Copy link
Member Author

jneira commented Oct 29, 2021

https://github.com/pingcap/dm/pull/1935/files

this is our repo's modification, for your reference

oh, i see, now it will cancel another workflows for the same branch, many many thanks

Without it will cancel other workflows for the same branch
Credits: 7b371e8b
@jneira jneira changed the title Add option to effectively cancel prev runs [skip circleci] Add option to effectively cancel prev runs Oct 29, 2021
@jneira
Copy link
Member Author

jneira commented Oct 29, 2021

@lance6716 dont want to abuse, but i see the workflow you changed was skipping the job if the pr modified the README.md, did you replace that with another way to do it?

@lance6716
Copy link

@lance6716 dont want to abuse, but i see the workflow you changed was skipping the job if the pr modified the README.md, did you replace that with another way to do it?

currently we can't skip jobs for only modifying README.md 😂 that didn't happen too often.

@jneira jneira added the merge me Label to trigger pull request merge label Oct 30, 2021
@github-actions
Copy link

We are gonna [skip circleci]

@mergify mergify bot merged commit ebe1d92 into master Oct 30, 2021
pepeiborra pushed a commit that referenced this pull request Nov 2, 2021
* Add option to cancel prev runs

Without using skip-duplicate-actions which is failing due to github access permission. 
See fkirc/skip-duplicate-actions#103

* Cancel by branch *and* workflow

Without it will cancel other workflows for the same branch
Credits: 7b371e8b

* Disable cancelling other runs

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants