Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use ruleset to enable branch protections #2081

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Conversation

minrk
Copy link
Member

@minrk minrk commented Oct 8, 2024

should avoid issues with the default branch not existing, which was required for the edit_protection API.

I believe this is the equivalent ruleset. I've run it against scifem-feedstock, so you can see the resulting rule here.

closes #2080

should avoid issues with the default branch not existing
Copy link
Member

@beckermr beckermr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use requests here instead of relying on a private attribute in pygithub.

@minrk
Copy link
Member Author

minrk commented Oct 8, 2024

OK. That's what I started to do, but this pattern is already used a number of times in this file already so I copied what's there.

I'll be afk until tonight if you want to take over and finish it before then.

@beckermr
Copy link
Member

beckermr commented Oct 8, 2024

Ok then it doesn't make sense to fix it here. Sigh. Lgtm.

@beckermr beckermr merged commit ef3d75f into conda-forge:main Oct 8, 2024
2 checks passed
@minrk minrk deleted the ruleset branch October 8, 2024 14:16
@minrk
Copy link
Member Author

minrk commented Oct 9, 2024

fwiw, I proposed making .requester a public API, which I think it should be anyway: PyGithub/PyGithub#3056

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.

applying branch protection fails with 404 before default branch exists
2 participants