AaronBallman wrote:

CC @rnk because of the concerns raised around our existing AI policy (I know 
Reid is working on updates, so not expecting you to review anything here, just 
for awareness).

> It matters because the PR description becomes the commit message.

FWIW, I agree that the PR summary doesn't really match our developer policy on 
[commit messages](https://llvm.org/docs/DeveloperPolicy.html#commit-messages). 
In particular:

> The body should be concise, but explanatory, including a complete rationale. 
> Unless it is required to understand the change, examples, code snippets and 
> gory details should be left to bug comments, web review or the mailing list.

Also:

> On nurturing new contributors — I don't disagree in principle, but the base 
> rate has changed.

I share this frustration and the concerns about the burdens our AI policies put 
on maintainers in particular. However, I think @nickdesaulniers was correct to 
push back on the original tone. Our [Code of 
Conduct](https://llvm.org/docs/CodeOfConduct.html) pushes us towards being 
careful in the words we choose, so that's worth keeping in mind for next time. 
Thank you for correcting that with more clarification; it's good that you 
raised your frustrations in a more productive way because public statements 
help folks like Reid when working on policies.

https://github.com/llvm/llvm-project/pull/209279
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to