alias [ci skip] to [skip ci]

[sethvargo]

I don't know how many times I accidentally kick off a build because I add [skip ci] instead of [ci skip] to my commit message.

It'd be nice if Travis honored both, and I think it would save you guys some builds 😄

[rubiii]

awesome proposal ✨ i always have to think twice before writing [ci skip] because to me it feels backwards.

[sarahhodne]

The thought behind it is that you can send commands to the CI server. Right now, the only command available is skip. Say we add a feature that lets you pay for a build to be pushed to the front of the queue (this has been discussed and won't be added, I'm just using it as an example). The command might be runquickly or something. You would then do [ci runquickly].

I'm currently undecided on whether adding the alias is a good idea or not, but hopefully the above will help you remember what order they are in now. The change itself for this is relatively simple, a regexp has to be updated in approval.rb.

[sethvargo]

I'm thinking more in terms of "English" I want to "skip ci" in my workflow.

[rubiii]

@henrikhodne i never thought about it as sending commands to ci, but now the naming makes sense.
i would still vote for [skip ci] though.

[rubiii]

@henrikhodne is the "ci" prefix also supposed to serve as some kind of "travis namespaces inside
commit messages" or is it just about the idea of sending commands?

[tjmcewan]

👍

[sarahhodne]

Four months later, I can say I'm not really opposed to this change. If anyone wants to send a PR for it, the code is at https://github.com/travis-ci/travis-core/blob/master/lib/travis/requests/services/receive/push.rb#L63 and https://github.com/travis-ci/travis-core/blob/master/lib/travis/model/request/approval.rb#L51.

[tjmcewan]

@henrikhodne how would feel about a new class that unifies that interface? you'd call it like, say, this: Travis::Command.new(commit['message']).skip?

[roidrage]

Fixed by travis-ci/travis-core#216.

[mathieujobin]

maybe not the right place to ask this
but can this [ci skip] be placed on a PR message so we can open a PR for review without the double-builds ? I need it per PR, not global, thats why

thanks

[tyru]

[skip ci] alias doesn't seem to work now?
In this repo, [skip ci] is included in commit message
but Travis CI Build was fired. vim-jp/vital.vim#380

[tyru]

I'm sorry, I was wrong.
As @mathieujobin said, merge commit was skipped but each message has no [skip ci] string, build was fired.