How about "Subject line should contain ': ' followed by a capitalized verb, and end in '.': "? Shorter and more info. On the other hand: there are more rules than just these 3 so I think after this change the next person is going to show up and say it's confusing :) With reason; which is exactly why the link to the code conventions is there, to take away confusion. I.e. I'd say that either this should have the complete explanation including examples, or else should be left as-is.

there are more rules than just these 3 so I think after this change the next person is going to show up and say it's confusing

Yes indeed, I didn't consider describing the other cases as they did not bothered me.

I'd say that either this should have the complete explanation including examples, or else should be left as-is.

A third alternative could be Subject line doesn't match "^[^!]+: [A-Z]+.+ .+\.$": it's concise, and not misleading. The code convention link at the end of a failed run already provides the explanation and examples. I would also add to this document that the sentence has to be capitalized (that was not obvious to me)

Showing the regex is indeed a good idea imo. Still confusing if you don't know regex, but at least then you can use one of the many online regex websites to verify.

Yeah, probably a good idea to actually list the regex, so that we don't run into this problem again (error message emitted when you think you did everything correctly).

@titouanc if you have the time could you update this to show the regex? This keeps popping up :)

@stinos Thank you for the heads up.

I amended my commit, and rebased on top of master

Awesome, looks good.

Thanks! Merged in d334d78