> On Feb 24, 2017, at 6:43 AM, Alexey Dokuchaev <da...@freebsd.org> wrote: > > On Fri, Feb 24, 2017 at 08:22:21AM -0600, Eric van Gyzen wrote: >> By the way, have you considered adding a Herald rule on Phabricator? >> Phab is not perfect, but a lot of folks use it, and the Herald rules >> are a very effective way to watch them play in one's area. > > +1, we need more Herald rules to make Phabricator more efficient. Of > course there are #manpages or #network groups, but often it is a bit > hard to come up with whom you'd want to place on the review list (like > if MAINTAINERS does not mention who's responsible and/or willing to > review the code you touch, and obviously you don't want to annoy some > random people just because you think they are technically qualified > enough or have required domain knowledge). Sending "Hey check this out > guys, Dxxxx" emails on e.g. -hackers@ is not a good option either (as > it would annoy even more people).
Yes, I have been looking into Herald(/MAINTAINERS integration): https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217316 <https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217316> . I’ve put up a series of rules for contrib/netbsd-tests and *.test.mk, so I’ll at least be pinged when changes are made to these areas. Thanks, -Ngie
signature.asc
Description: Message signed with OpenPGP