Hi,

I've added few labels related to PR:

PR: needs more work (author's turn)
PR: waiting for Travis to finish

and then we can use them in a query. In particular, to only see PRs that

* are open
* pass all tests
* do not have the "PR: needs more work (author's turn)" label
* sort by recently updated

do:

https://github.com/sympy/sympy/pulls?utf8=%E2%9C%93&q=is%3Apr+is%3Aopen+status%3Asuccess+sort%3Aupdated-desc+-label%3A%22PR%3A+needs+more+work+%28author%27s+turn%29%22+

Those are all PRs that either can be merged or reviewed. If after the
review the author needs to do more work, just add the label "PR: needs
more work (author's turn)".

Then, when you start your reviewing session, start by looking at the
PRs where it was author's turn, sort by recently updated:

https://github.com/sympy/sympy/issues?q=is%3Aopen+label%3A%22PR%3A+needs+more+work+%28author%27s+turn%29%22+sort%3Aupdated-desc

and remove the label to PRs where the author did the work.

I started using this workflow, it seems it is a lot more manageable.
Let me know what you think. We should refine this and document it.

Ondrej

-- 
You received this message because you are subscribed to the Google Groups 
"sympy" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sympy.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sympy/CADDwiVAMnWPw81ow04n9BmAArMYfb1_9EAx5WAFWNLffn1xQAw%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to