Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Add ability to append [ci skip] to commit messages #1011

Merged
merged 1 commit into from
Mar 19, 2018
Merged

Conversation

squaremo
Copy link
Member

@squaremo squaremo commented Mar 16, 2018

Appending [ci skip] to a commit message will instruct CI to skip the
build for that commit, at least in these systems:

So: I've added a flag which switches that will do exactly that, to all
commits fluxd creates. To account for other CI systems that have a
similar mechanism but don't use that convention, the flag
--git-ci-skip-message lets you supply the text to append.

Resolves #856.

Copy link
Contributor

@samb1729 samb1729 left a comment

Choose a reason for hiding this comment

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

One obviousness remark for you to decide upon. Otherwise LGTM

@@ -145,6 +148,12 @@ func main() {
}
}

switch {

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

Appending `[ci skip]` to a commit message will instruct CI to skip the
build for that commit, at least in these systems:

 - https://circleci.com/docs/1.0/skip-a-build/
 - https://docs.travis-ci.com/user/customizing-the-build/#Skipping-a-build
 - https://docs.gitlab.com/ee/ci/yaml/#skipping-jobs

So: I've added a flag which switches that will do exactly that, to all
commits fluxd creates. To account for other CI systems that have a
similar mechanism but don't use that convention, the flag
`--git-ci-skip-message` lets you supply the text to append.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants