mbien commented on PR #8959:
URL: https://github.com/apache/netbeans/pull/8959#issuecomment-3508462674

   > @mbien that's a lot of text to reiterate what we already discussed on 
Slack! 😄 
   
   @neilcsmith-net if i would have more time I would have written a shorter 
reply ;)
   
   I could have said that merged PRs without review may cause revert PRs 
without review, but this would be the wrong mindset IMO and unnecessarily 
confrontational. Its more constructive to talk about practices which are there 
to avoid reverts in the first place - even though it requires more text. (and I 
did also [at no point 
claim](https://github.com/apache/netbeans/pull/8959#issuecomment-3448486793) 
that merges would _require_ review)
   
   Sorry to have derailed this PR after getting pinged, but I simply don't 
think its beneficial to point out that this repo is run in "in post-merge 
approval mode! ", while it is easily verifiable for anyone that in fact all 
noteworthy/risky PRs do get reviewed. [NB 28 list of unreviewed 
merges](https://github.com/apache/netbeans/pulls?q=is%3Apr+milestone%3ANB28+is%3Amerged+review%3Anone+).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to