On 11/15/18 7:19 PM, Stefan Sperling wrote:
So please don't stop voicing your opinions.
For what it's worth, I would prefer to enforce the rule as well. Even if it does not seem like much effort to explain this workflow to new users, and to ask people to think about the +3 before merging: these things add up, leave room for mistakes and scale badly (as already pointed out in this thread).
It's the same with CI that builds code and runs test cases - of course we could trust everybody to do this on their own before publishing patches, but that is not as good as checking it automatically and having proof that the tests really ran through before merging.
Best regards, Oliver