Re: [Action Required] Stale Reviews

2016-07-12 Thread Jake Farrell
Seems like the discussion has died down on this, unless anyone objects i'll run through and close out all marked pull requests in this doc later today -Jake On Sat, Jul 9, 2016 at 9:43 PM, Joris Van Remoortere wrote: > I've discarded almost all reviews older than 3

Re: [Action Required] Stale Reviews

2016-07-09 Thread Joris Van Remoortere
I've discarded almost all reviews older than 3 months. Please do feel free to re-open them and work with a committer if you feel your review is still viable and want to continue making progress on it. I will be going through the remainder more carefully. Let's try to discard reviews when they

Re: [Action Required] Stale Reviews

2016-07-06 Thread Joseph Wu
On a related note, we will also be looking at the (usually neglected) GitHub PRs. We've accumulated ~50 of them over time. After making a quick scan of the list, it turns out we can close a majority of these PRs by either directly closing the non-issues, or by committing the small documentation

Re: [Action Required] Stale Reviews

2016-07-03 Thread Alex Rukletsov
Joris, could we punt on this until after 1.0? Right now people focus on polishing things for the release and I would like to avoid any distractions. On Wed, Jun 29, 2016 at 8:25 PM, Joris Van Remoortere wrote: > Your suggestion generally encompasses the spirit of what we will

Re: [Action Required] Stale Reviews

2016-06-29 Thread Joris Van Remoortere
Your suggestion generally encompasses the spirit of what we will do after we've given the community time to act on their own. The reason we will likely go through them manually is that there will be some patches that don't apply but for which the contributor would still like to resume work.

Re: [Action Required] Stale Reviews

2016-06-29 Thread Tomek Janiszewski
How about running CI on all reviews. If patch is stale it probably can't be applied, CI will post bad patch and if nobody do any action on that review we can close it. śr., 29.06.2016, 18:26 użytkownik Joris Van Remoortere napisał: > Hello developers, > > Over the last year

Re: [Action Required] Stale Reviews

2016-06-29 Thread Cong Wang
On Wed, Jun 29, 2016 at 9:25 AM, Joris Van Remoortere wrote: > Hello developers, > > Over the last year we've accumulated a significant review backlog. Over the > past month it has been floating around ~600 reviews. Why do you keep thinking the problem is on this side rather

[Action Required] Stale Reviews

2016-06-29 Thread Joris Van Remoortere
Hello developers, Over the last year we've accumulated a significant review backlog. Over the past month it has been floating around ~600 reviews. It would be of great help if you could look through your personal list (Dashboard -> Outgoing -> Open) and identify reviews that are *no longer