eirikbakke commented on code in PR #9451: URL: https://github.com/apache/netbeans/pull/9451#discussion_r3442046961
########## .github/PULL_REQUEST_TEMPLATE.md: ########## @@ -13,7 +13,12 @@ By opening a pull request you confirm that, unless explicitly stated otherwise, - are all your own work, and you have the right to contribute them. - are contributed solely under the terms and conditions of the Apache License 2.0 (see section 5 of the license for more information). -Please make sure (eg. `git log`) that all commits have a valid name and email address for you in the Author field. +Commit messages and PR description: + + - Please make sure (eg. `git log`) that all commits have a valid name and email address for you in the Author field. + - LLM assisted commits should be attributed with a `Assisted-by: MODEL_NAME MODEL_VERSION` line appended to the commit message. + - Please mention coding assistance in the PR description too (eg. by adding the same `Assisted-by` line from above) + - Please describe the changes in your own words focusing on the important aspects Review Comment: I agree with that. At the very least the PR description should be human-written, for proving understanding of the changes, and to provide some signal to reviewers about the author's commitment to see the PR through. Neil's wording would work, I think. (Including the exclamation mark.) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
