Please do not close this issue until this is either fixed or a valid workaround is offered and confirmed to work

The same issue has been reported several times (https://github.com/microsoft/azure-pipelines-agent/issues/2441 and https://github.com/microsoft/azure-pipelines-agent/issues/858) but the issue is still there.

According to https://developercommunity.visualstudio.com/content/problem/714477/pr-commit-with-skip-ci-in-message-still-triggers-a.html this is by design:

PRs currently do not honor [skip ci] keywords. This is to ensure that pipeline policies on PRs cannot be circumvented by including [skip ci] in the PR commits.

However that design is flawed because:

it goes against industry standard, is surprising, and doesn’t play well with other CI

see https://github.com/Microsoft/azure-pipelines-agent/issues/858#issuecomment-340012115

Basically most of the others CI use [ci skip] or [skip ci] (e.g. Travis, CircleCI, Concourse) Some dependencies use [ci skip] as a pattern to skip automatically versioning from their CI, why not make VSTS also accept it?

it’s not what your users want

everyone is complaining about this:

• https://github.com/microsoft/azure-pipelines-agent/issues/2441
• https://github.com/microsoft/azure-pipelines-agent/issues/858
• https://github.com/Azure/azure-sdk/issues/474
• https://developercommunity.visualstudio.com/content/problem/714477/pr-commit-with-skip-ci-in-message-still-triggers-a.html
• https://developercommunity.visualstudio.com/idea/420289/add-support-ci-skip-and-azure-skip-to-azure-pipeli.html

eg:

This behavior is undesirable, regardless of whether it was by design or by oversight […]. But even though that commit message specifies [skip ci] or NO_CI, CI build triggers are still fired. It makes no sense why this commit-via-PR shouldn’t have to play by the same rules as other commits.

or this: https://github.com/Microsoft/azure-pipelines-agent/issues/1270#issuecomment-486787843

@kelliejos Why do you think that you know what end-users want? I want use skippers for all builds. Thanks, but for GitHub PRs, it looks really weird when [CI skip] is ignored by Azure Pipelines but honoured by Travis CI and AppVeyor.

it invites dangerous workarounds

• this causes workarounds like this:

The workaround for me is painful, and risky: 1). disable all policies on master branch (several dozen), 2). commit changes directory to master without benefit of a code review, which is why I want the PR!, and 3). re-enable policies on master branch.

it slows down development

because CI runs regardless what we specified

it’s illogical

It is very strange that this feature skips CI build if any commit in the history has one of the indicator strings. This means, for example, that backporting a commit with [ci skip] onto a release branch skips CI for that branch even if the following 10 commits are un-skipped. Travis only skips if the most recent commit contains the skip indicator.

the rationale given makes no sense

This is to ensure that pipeline policies on PRs cannot be circumvented by including [skip ci] in the PR commits.

We don’t need this babysitting; it’s a PR, it still needs to be merged and it’s trivial for a reviewer to check whether the CI was run or not (or whether last commit message message contained [ci skip]; reviewer can always request to change the last commit msg if he disagrees.

There’s still a reviewer involved. Second guessing humans isn’t helpful. Example: on a repo where I’m a maintainer I made an edit to a plan text file. I want to send it via a PR so it’s visible to other maintainers that I’m making a change (pushing directly to master is considered bad form). But I don’t need to trigger hours of CPU time worth of CI builds. I’m 99.99% sure I’m not breaking anything. And in the very unlikely case I did break something, I can just go back and fix it or revert the change.

it’s not even documented

https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github?view=azure-devops&tabs=yaml#skipping-ci-for-individual-commits makes no mention of it not working for PR’s

it will save you money and is good for the environment

…by avoiding wasting CI cycles. I mean, who doesn’t want to save the environment 😉

---

proposal

either fix the behavior of [ci skip] or provide a simple way to enable honoring [ci skip]for PR commit messages in azure pipelines, eg allow_pr_ci_skip: true