Alexis, I agree, the revert use case should be supported more, often I run into the 50 char rule just committing – so we can push this back to 40 char and count them before we commit, or remove the restriction – I am OK either way.
/michael From: [email protected] [mailto:[email protected]] On Behalf Of Alexis de Talhouët Sent: Monday, November 27, 2017 15:13 To: Mike Evans via onap-discuss <[email protected]> Cc: [email protected] Subject: [onap-tsc] [Gerrit] Commit message restriction Hi ONAP devs, committers, PTLs, I want to start a thread wrt the commit message restriction; bellow my observations: - reverting a patch through gerrit UI often fails with 500 ERROR due to: commit subject >50 characters; use shorter first paragraph If we think about it, someone creates a patch with 50 characters as commit title, then someone wants to revert, it automatically force the user to revert using CLI because the reverts add “revert: “, which adds 9 characters, hence validation fails. This is somewhat annoying. - pasting a stacktrace as a commit message is kind of prohibited: too many commit message lines longer than 72 characters; manually wrap lines I understand as a community we want to foster developers to use common practices, but should we really enforced those? I worked in a couple of other communities that has a large pool of contributors and everything is working fine, without those mentioned two rules enforced. I tend to think it’s the commiter role to enforce best practices on commit msg within his project, it shouldn’t be enforced by the system itself. So I’d like to know devs/committers/PTLs feeling about this, and also ask the TSC whether this is sealed in rock or not. Thanks, Alexis This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy statement, you may review at https://www.amdocs.com/about/email-disclaimer <https://www.amdocs.com/about/email-disclaimer>
_______________________________________________ ONAP-TSC mailing list [email protected] https://lists.onap.org/mailman/listinfo/onap-tsc
