Okay, we have now master branch protection in place for NuttX GitHub repo :-)
I am sorry for any inconvenience and troubles caused in a short window of testing! If all works okay, please let me know, also if there are any other updates required! When all works as expected I will apply similar protections for nuttx-apps and nuttx-website repos :-) Thank you and sorry again for the trouble! :-) Tomek On Wed, May 7, 2025 at 1:57 PM Tomek CEDRO <to...@cedro.info> wrote: > > Alllright, so we merged that change and we have two problems. I have > already contacted Apache Infra for support :-) > > 1. strict:true makes all PR require to be rebased each time master is > updated. That puts additional work on committers and eats up precious > CI resources as we have seeral master merges daily. This should be set > to false by default. > > 2. required_signatures:true not only means git commit -s but > cryptographic signatures that are not required by our PR process. > > But we are locked out from merging the fix ourselves now. The fix is > here https://github.com/apache/nuttx/pull/16336 :-) > > This reference https://github.com/apache/infrastructure-asfyaml may be > updated with above hints / possible problems :-) > > Tomek > > On Wed, May 7, 2025 at 4:45 AM Nathan Hartman <hartman.nat...@gmail.com> > wrote: > > > > Looks good and looks like it has been merged! Thank you Tomek for tackling > > this!!! > > > > Cheers, > > Nathan > > > > On Tue, May 6, 2025 at 4:46 PM Tomek CEDRO <to...@cedro.info> wrote: > > > > > Thank you Nathan :-) > > > > > > I have talked with @Humbedooh on Slack, he said the config looks okay :-) > > > > > > I am not really sure about this "contexts" part, particularly the > > > exact naming convention in GitHub, so I am leaving this for now and > > > hope it wont break anything :-) > > > > > > contexts: > > > - Check > > > - Lint > > > - Build > > > > > > This means if Check, Lint, or Build fails then PR cannot be merged. > > > > > > Thank you :-) > > > Tomek > > > > > > On Tue, May 6, 2025 at 6:42 PM Nathan Hartman <hartman.nat...@gmail.com> > > > wrote: > > > > > > > > Hello Tomek :-) > > > > > > > > Thanks for doing that! > > > > > > > > I have found that the easiest/quickest way to talk to the Infra folks is > > > by > > > > Slack [1]. > > > > > > > > If you don't want to use Slack, you can write to Infra's users list > > > > (also > > > > see [1]). > > > > > > > > I recommend to try slack, but either way, give them the link to the PR > > > and > > > > ask nicely if they could verify it... > > > > > > > > [1] To contact Infra by Slack, see the section "Chat" at > > > > https://infra.apache.org/contact.html > > > > > > > > Hope that helps, > > > > Nathan > > > > > > > > On Tue, May 6, 2025 at 9:21 AM Tomek CEDRO <to...@cedro.info> wrote: > > > > > > > > > Hello world :-) > > > > > > > > > > I found a free moment to setup github master branch protections as > > > > > discussed before, for now PR is in Draft mode for careful review. Who > > > > > can we contact from Apache Admins to ask to verification? > > > > > > > > > > https://github.com/apache/nuttx/pull/16324 > > > > > > > > > > * We do not have "Settings" tab in the Apache's owned repository, thus > > > > > we need to update .asf.yaml file with repository settings. > > > > > * No direct push to master branch is possible. > > > > > * Require status checks to pass before merge. > > > > > * Setup reviews parameters. > > > > > * Require signatures. > > > > > * Require conversation resolution. > > > > > * Not sure if we want to enforce "linear history"? > > > > > * If all is fine here I will do the same update for nuttx-apps repo > > > > > :-) > > > > > * I am not really familiar with all these settings, please review if > > > > > this is what we want/need. > > > > > * Particularly the "checks" needs verification if these are valid > > > > > names (@lupyuen?). > > > > > > > > > > Reference: https://github.com/apache/infrastructure-asfyaml. > > > > > > > > > > Thank you :-) > > > > > Tomek > > > > > > > > > > -- > > > > > CeDeROM, SQ7MHZ, http://www.tomek.cedro.info > > > > > > > > > > > > > > > > > -- > > > CeDeROM, SQ7MHZ, http://www.tomek.cedro.info > > > > > > > -- > CeDeROM, SQ7MHZ, http://www.tomek.cedro.info -- CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
