On 2015-09-13 7:57 PM, Anthony Ramine wrote:
S-needs-rebase and S-awaiting-review should be mutually exclusive though, right?

Le 14 sept. 2015 à 01:49, Josh Matthews <j...@joshmatthews.net> a écrit :

I will point out that any PR with S-awaiting-review should not be closed; that 
means that the failure is on our end since new code has been pushed since the 
last time someone reviewed it.


No; S-needs-rebase is applied automatically to PRs when homu posts a Merge Conflict comment. It intentionally doesn't remove the S-awaiting-review label.
_______________________________________________
dev-servo mailing list
dev-servo@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-servo

Reply via email to