I think that it would be nice if our highfive had the feature that Rust's does (IIRC) where it can assign a reviewer based on area of interest.
If you're referring to the governance list on the wiki (https://github.com/servo/servo/wiki/Governance#super-reviewers), it's not outdated, but just a different thing. Super reviewers are the people who should be involved if there is a substantial design change proposed for an area. - Lars On Wed, Mar 9, 2016 at 12:23 PM, Keith Yeung <kungfukeit...@gmail.com> wrote: > Should we have a list of people who are familiar with a certain crate > within Servo? I think the wiki has a list, but it seems oudated. > On Mar 9, 2016 11:16 AM, "Jack Moffitt" <j...@metajack.im> wrote: > >> On Wed, Mar 9, 2016 at 8:25 AM, Josh Matthews <j...@joshmatthews.net> >> wrote: >> > Hi reviewers of Servo! As you've noticed, we recently started assigning >> > reviewers to new PRs by random selection. Sometimes this means that a >> > reviewer ends up assigned to a PR which they don't feel comfortable >> > reviewing, which is totally fine! When that happens, please redirect the >> > review to somebody else more appropriate rather than just removing >> yourself >> > as the assignee. >> >> I would also encourage those of you getting PRs outside your comfort >> zone to at least glance at the PR and maybe try and do what you can. >> You'll learn something new, expand your comfort zone, and probably >> catch some things other reviewers didn't. I often find that when I >> review code I'm not as comfortable with, I'm usually much more careful >> and catch more undocumented assumptions. >> >> jack. >> _______________________________________________ >> dev-servo mailing list >> dev-servo@lists.mozilla.org >> https://lists.mozilla.org/listinfo/dev-servo >> > _______________________________________________ > dev-servo mailing list > dev-servo@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-servo _______________________________________________ dev-servo mailing list dev-servo@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-servo
