[cfe-dev] [llvm-dev] Phabricator Maintenance
Johannes Doerfert via cfe-dev
cfe-dev at lists.llvm.org
Thu Jun 25 09:33:27 PDT 2020
On 6/25/20 11:33 AM, James Y Knight wrote:
> On Thu, Jun 25, 2020 at 11:24 AM Johannes Doerfert via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>> On 6/24/20 3:41 PM, Chris Lattner via llvm-dev wrote:
>> When I looked into this possibility (GitHub PR) when we merged MLIR in the monorepo: just Herald rules was already a blocker and I couldn't find a replacement readily available on GitHub. The handling of stack of revisions on GitHub was another problem mentioned.
>> Fair enough - I haven’t investigated the technical issues with adopting GitHub PRs for LLVM. My experience with the Swift community has been extremely positive though, I’d be surprised if there is something particularly crazy that only LLVM needs that GitHub hasn’t encountered. That said, again, I don’t have any practical experience with this part of the stack,
>> I am very much relying on Herald rules to filter patches (=PRs).
>> 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.
That would be great :)
> 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.
> ~ Johannes
>> LLVM Developers mailing listllvm-dev at lists.llvm.orghttps://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>> LLVM Developers mailing list
>> llvm-dev at lists.llvm.org
More information about the cfe-dev