On Wednesday, August 21, 2024 at 12:56:15 AM UTC+9 julian...@fsfe.org wrote:

implies that "disputed" label is only for a PR that has positive review 
from a reviewer but another still objects it. I think adding "disputed" 
label prematurely is not appropriate. Then who can remove the "disputed" 
label in this case?


That's not how I interpret the policy we have. I think the idea is to first 
use the "needs whatever" labels and when no agreement can be reached, then 
"disputed" can be set. If we had to wait for a "positive review", then 
there would be a race condition of sorts: You'd have to be fast so the 
release manager does not pick the PR up while the positive review label is 
set.


OK. That concern is reasonable. Then at least there should be explicit 
issues (expressed in comments) on which the author and the negative 
reviewer disagrees before "disputed" label is added.  


 

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sage-devel/705b89c8-02a5-4c2d-9b60-a895b33c0932n%40googlegroups.com.

Reply via email to