[GitHub] [netbeans] mbien commented on pull request #4785: enable branch protection for master.

[GitBox <gi...@apache.org>]

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

   The main reason why i opened this draft is because branch protection came up in a discussion recently. So i thought before I forget about it, I read the ASF doc and check what is possible.
   
   
   > > enable force push protection for delivery too?
   > 
   > Please don't! `delivery` is a transient branch that we do occasionally need to force push into a suitable state.
   
   i had the same feeling that is why I left it commented out.
   
   
   > > add reviewer requirement for master?
   > 
   > anyone trusted to merge should be able to make the judgement on what level of review is required for any change. And anyway, code can always be vetoed and/or reverted after the fact if necessary.
   
   sure
   
   so essentially we can reduce it to `master: {}` and leave everything else unconfigured/disabled. Although having green checkmarks would be a interesting requirement for later.


-- 
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: notifications-unsubscribe@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

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