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
