Re: Disable force push on default branch

2025-05-08 Thread Konrad Windszus
Thanks for the input. Removing the protection is it straightforward and doesn’t require ASF INFRA involvement. You just temporarily change the asf.yaml again (as outlined in https://github.com/apache/infrastructure-asfyaml?tab=readme-ov-file#branchpro). Meanwhile the GitHub branch protection is

Re: Disable force push on default branch

2025-05-02 Thread Robert Munteanu
On Wed, 2025-04-30 at 11:51 -0700, Eric Norman wrote: > Also, we may need to think of the process to handle the exceptions to > that > rule if a force push is temporarily necessary for some special case. That is a good point. I think the the only way out is to ask ASF Infra to remove branch protec

Re: Disable force push on default branch

2025-05-02 Thread Robert Munteanu
On Wed, 2025-04-30 at 15:31 +0200, Konrad Windszus wrote: > WDYT about adding this to all our .asf.yaml’s? +1 Robert

Re: Disable force push on default branch

2025-05-02 Thread Robert Munteanu
On Fri, 2025-05-02 at 10:00 +, Stefan Seifert wrote: > not sure if we have modules where the default branch is not called > master. I strongly doubt it, the repo scripts would break otherwise. Robert

RE: Disable force push on default branch

2025-05-02 Thread Stefan Seifert
yes, we should disable force push for the default branch (master) everywhere by default. not sure if we have modules where the default branch is not called master. regarding erics remark: > It would be better if that could somehow be made the default for the sling > organization instead of havin

Re: Disable force push on default branch

2025-05-02 Thread Julian Sedding
+1 from me. Eric raises a valid issue rgd. a process around it in case due to security reasons. Presumably, we could change the .asf.yaml file (temporarily) should a situation arise that requires a force push? Regards Julian On Wed, 30 Apr 2025 at 20:52, Eric Norman wrote: > > I don't think I ha

Re: Disable force push on default branch

2025-04-30 Thread Eric Norman
I don't think I have ever done a force push on purpose so I had to refresh my memory on what the implications were for such things. I found a summary snippet at https://github.com/orgs/eclipse-csi/discussions/7 that clarified the risks in my mind so maybe that info is helpful for others? Force-pu