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 PR builds via commit message #474

Closed
mikeharder opened this issue Jul 9, 2019 · 9 comments
Closed

Skip PR builds via commit message #474

mikeharder opened this issue Jul 9, 2019 · 9 comments
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Comments

@mikeharder
Copy link
Member

We should try to add a mechanism to allow skipping PR builds via a commit message, to avoid running unnecessary builds when a change is known to be very low-risk. DevOps has a built-in feature to do this, but it only works for CI builds, not PR builds:

microsoft/azure-pipelines-agent#858 (comment)
microsoft/azure-pipelines-agent#1270 (comment)

We might be able to implement this using custom code which inspects the commit message and either stops the build or sets a flag which tells other steps to no-op.

@mikeharder mikeharder added the EngSys This issue is impacting the engineering system. label Jul 9, 2019
@mikeharder mikeharder self-assigned this Jul 9, 2019
@pllim
Copy link

pllim commented Sep 10, 2019

Please, please, make this happen. I have been waiting for this.

@jhunkeler
Copy link

Yes, please implement this.

@mihaimircea
Copy link

This would be useful.

@AutumnEvans418
Copy link

We really need this as well.

@kurtzeborn kurtzeborn added the Central-EngSys This issue is owned by the Engineering System team. label Sep 28, 2020
@StephenHodgson
Copy link

any update on this?

@weshaggard
Copy link
Member

No we don't have much of an update here. We do have some potential ideas to get some level of this functionality but unfortunately we cannot do this completely on our side it would require changes from Azure DevOps which I don't see happening.

cc @benbp

@mscheltienne
Copy link

I think MNE has a workaround that enables [skip azp] in a PR commit message to skip the Azure CI: https://github.com/mne-tools/mne-python/blob/948f0dc2745ba44d5add4faabc4090075bf1a770/azure-pipelines.yml#L16-L32

@weshaggard
Copy link
Member

Thanks for the pointer. This is a similar workaround that we have been considering but just haven't implemented it yet.

Copy link

Hi @mikeharder, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 20, 2024
@github-project-automation github-project-automation bot moved this from 🤔 Triage to 🎊 Closed in Azure SDK EngSys 🤖🧠 Mar 20, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.
Projects
Archived in project
Development

No branches or pull requests

9 participants