[llvm] [GitHub] Allow shortcut for "introductory issue" and request linking to issue in PR (PR #84635)

Danny Mösch via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 16:03:17 PDT 2024


SimplyDanny wrote:

That seems to be handled differently in different parts of the project. In the clang-tidy area, for instance, I often see the "just create a PR" comments to avoid bothering someone to assign people. Especially for tickets lying around untouched for a while, this is a typical advice. Two people working on the same topic is unlikely then. That's what brought me to this update.

I get your points, but also wonder if not both approaches are valid. The issues we are talking about are already tagged as "good first issue", so more experienced contributors would rather leave them for newcomers anyway. Now, asking for assignment is of course valid. Then the matter is clear. But if nobody got assigned and someone prepared a PR already, why not accept it as well? By referencing the issue in the PR, the PR appears in the issue and that would serve as an assignment. Therefore, I added this "mention the issue in the PR description" part.

https://github.com/llvm/llvm-project/pull/84635


More information about the llvm-commits mailing list