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 CI requests #1148

Closed
jakirkham opened this issue Jul 25, 2016 · 19 comments · Fixed by #27765
Closed

Skip CI requests #1148

jakirkham opened this issue Jul 25, 2016 · 19 comments · Fixed by #27765

Comments

@jakirkham
Copy link
Member

Trying to keep track of these in one place.

  1. Make [ci skip] work in PRs for CircleCI. ( ref )
  2. Have [circle skip] to skip CircleCI specifically. ( ref )
  3. Have [travis skip] to skip Travis CI specifically. ( Add a [skip travis] command travis-ci/travis-ci#5032 )
@jakirkham
Copy link
Member Author

Appears CircleCI now respects [ci skip] on pull requests, but not [skip ci] yet.

@smithsp
Copy link
Contributor

smithsp commented Dec 11, 2017

What is the placement of these strings? In a commit message, or in a pull request title? What about [skip appveyor]?

@CJ-Wright
Copy link
Member

In the commit message.

@CJ-Wright
Copy link
Member

I don't know if appveyor has a skip. @jakirkham does appveyor have a skip?

@smithsp
Copy link
Contributor

smithsp commented Dec 11, 2017

@CJ-Wright
Copy link
Member

CJ-Wright commented Dec 11, 2017

Not the same, that skips the CI forever. We just want to skip the CI temporally. (eg the only change you made was removing a comment). The CIs will all run as proper when we go to merge (or one puts in another commit without those in the message)
Edit: I take that all back, it seems that there is an appveyor skip, sorry (I was looking at the wrong section)

@CJ-Wright
Copy link
Member

Just for ease of reference [skip appveyor] is listed in the Readme as @smithsp found.

@smithsp
Copy link
Contributor

smithsp commented Dec 11, 2017

It is odd that I added those strings in my commit message, but #4540 still triggered circleci and appveyor builds.

@CJ-Wright
Copy link
Member

It seems they can be a bit fickle.

@jakirkham
Copy link
Member Author

Probably should add that the ones listed in the OP are all ones that don’t work, but we or others have requested the CIs add them.

@smithsp
Copy link
Contributor

smithsp commented Jan 12, 2018

Note that I found https://www.appveyor.com/docs/how-to/filtering-commits/ which indicates the [skip appveyor] (or [skip ci]) directive must be in the git commit title line for appveyor. My latest commit in #4560 seems to confirm this.

@jakirkham
Copy link
Member Author

That's correct. Also issue ( appveyor/ci#1865 ) may be of interest w.r.t. AppVeyor's commit filtering.

@croth1
Copy link

croth1 commented Mar 17, 2019

Does anybody have an overview of the current status?

I think it would be interesting to collect all of these in the documentation: http://conda-forge.org/docs/maintainer/infrastructure.html#skipping-ci-builds

It would be also interesting to have instructions to skip mac and windows but not linux, etc.

@ceholden
Copy link
Contributor

ceholden commented Mar 28, 2019

Just came across an update for Azure on their issue tracker (post) -- they've just enabled skipping for commit messages with [skip ci] and a few other tags

[EDIT]: Doesn't look like it's working yet! https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=25184

@kynan
Copy link
Contributor

kynan commented Apr 10, 2021

Is there any update on this? The docs mention [ci skip] ***NO_CI*** as the tag to include, but that doesn't seem to have any effect for Azure.

Can I still safely push a commit without triggering a new build and new packages to be published?

@kynan
Copy link
Contributor

kynan commented Apr 10, 2021

Ideally, I'd like the linter to still run on my recipe, but be sure that no builds will be triggered.

@beckermr
Copy link
Member

It doesn't work for azure pull requests but it does work on the master branch of the feedstock.

@kynan
Copy link
Contributor

kynan commented Apr 10, 2021

OK. Just to make sure I understand correctly: when I want to skip CI I should be committing directly commit to master and not make a PR?

@beckermr
Copy link
Member

Nope. Still use prs. It is just that ci skip in prs may not work. However prs do not upload packages so this isn't an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

7 participants