Re: [dev-servo] Please don't leave PRs without an assignee

2016-03-09 Thread Lars Bergstrom
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 dif

Re: [dev-servo] Please don't leave PRs without an assignee

2016-03-09 Thread Keith Yeung
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" wrote: > On Wed, Mar 9, 2016 at 8:25 AM, Josh Matthews > wrote: > > Hi reviewers of Servo! As you've noticed, we recently

Re: [dev-servo] Please don't leave PRs without an assignee

2016-03-09 Thread Jack Moffitt
On Wed, Mar 9, 2016 at 8:25 AM, Josh Matthews 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

[dev-servo] Please don't leave PRs without an assignee

2016-03-09 Thread Josh Matthews
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 so