Skip to content

Conversation

hoheinzollern
Copy link
Member

No description provided.

@CohenCyril
Copy link
Member

@erikmd @hoheinzollern any clue why the master branch is protected?

@hoheinzollern
Copy link
Member Author

No clue

@erikmd
Copy link
Collaborator

erikmd commented Apr 28, 2025

Hi @hoheinzollern @CohenCyril thanks for the ping!

I am also puzzled to see Merging is blocked - Cannot update this protected ref.

I'll investigate ASAP

@erikmd
Copy link
Collaborator

erikmd commented Apr 28, 2025

The current setting that is applied since a long time, is that master is protected so that 1. one cannot force-push master (which is a fairly standard setting) and 2. only docker-mathcomp's Maintainers team can merge a PR in docker-mathcomp (as they automatically belong to the bypass list).

But I see this in the Rulesets settings:

2025-04-29_00-04-11_Screenshot_Bypass_list

So I suspect that someone in the math-comp admins deleted or renamed a team.

Let me know if you know which math-comp GH team could be added in this bypass list, instead of Team @null (no member)

Meanwhile, I added both of you @CohenCyril @hoheinzollern with the Admin (> Maintain) role, cf. this.

@hoheinzollern
Copy link
Member Author

I can see that you added me @erikmd, thanks, should I then self-merge?

@erikmd
Copy link
Collaborator

erikmd commented Apr 29, 2025

Yes, feel free! — so you'll be able to check that the merge/bypass permissions are fine with your account as well.

@hoheinzollern hoheinzollern merged commit ac3b402 into math-comp:master Apr 30, 2025
1 check passed
erikmd added a commit to rocq-community/docker-base that referenced this pull request May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants