[cfe-dev] [llvm-dev] Phabricator Maintenance

Nathan Froyd via cfe-dev cfe-dev at lists.llvm.org
Fri Jun 26 08:11:04 PDT 2020


On Thu, Jun 25, 2020 at 12:33 PM James Y Knight via cfe-dev <
cfe-dev at lists.llvm.org> wrote:

>
> On Thu, Jun 25, 2020 at 11:24 AM Johannes Doerfert via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>
>> We need some replacement, IMHO, as we have many more patches/commits than
>> swift and the mono-repo is more diverse (I would imagine).
>>
>
> Yes, but this part of the problem is trivial to solve. We can add a github
> workflow which triggers on pull-requests being uploaded or modified, which
> will @ people who ask to be, based on a configuration file matching commit
> paths/descriptions. There may even be something existing which does this,
> but even if not, it should be no more than a week to fully implement.
>
> If resolving this would actually let us migrate to GitHub, I'd volunteer
> to implement it myself, but it seems like there's other issues which are
> less trivial to solve that might also block it.
>
>
It is not quite a full replacement for Herald rules, but Github supports
labeling PRs depending on what files they modify:

https://github.com/actions/labeler

You can see it in action in the wasmtime repo, with its labels:

https://github.com/bytecodealliance/wasmtime/blob/main/.github/labeler.yml

and then using the subscribe-to-label action:

https://github.com/bytecodealliance/subscribe-to-label-action

people can get automatically CC'd to whatever areas they like.  The only
downside is that information lives in the repo itself, so people who want
to be notified have to submit "fixes" to the repo.

I know Herald lets you automatically add reviewers for given paths; it
seems like that functionality ought to be supportable through Github
actions as well.

-Nathan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20200626/590ab902/attachment.html>


More information about the cfe-dev mailing list