[PATCH] D146819: [llvm][github] Add good-first-issue comment to issues
Timm Bäder via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 31 22:30:23 PDT 2023
tbaeder marked 3 inline comments as done.
tbaeder added a comment.
In D146819#4233226 <https://reviews.llvm.org/D146819#4233226>, @aaron.ballman wrote:
> Thank you for working on this, I think it's a really good idea! One issue I see with it though is that the GitHub issues list is used by multiple projects (llvm, clang, lldb, libc++, etc) but the way in which you contribute to these projects differ. So I think these instructions will work great for most folks, but we may want to consider the other projects and whether we need to adjust any wording here accordingly. (Personally, I think any project within the monorepo that diverges from the usual policies sufficiently enough to cause problems like this should either consider moving out of the monorepo or changing their policies to be in line with the rest of the monorepo, but that's a much bigger discussion than this patch. I think this patch kind of demonstrates why those policy differences cause friction for new contributors.)
Right, I was thinking the script could be more sophisticated and e.g. check that any of the labels are of the "clang:" kind - and then post clang-specific instructions, and do the same for lld, libc++, etc. but that's just not an exact science.
If these instructions are a problem for any of the subprojects, we can still iterate on them. Should I maybe add someone from lld/lldb/libc++/etc. to the CC here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146819/new/
https://reviews.llvm.org/D146819
More information about the llvm-commits
mailing list