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

Try to force precommit-ci to send autoupdate PRs. #1325

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Feb 13, 2025

It has not in a loong time, but I'm not sure why.

It has not in a loong time, but I'm not sure why.
@Carreau Carreau added this to the 7.0 milestone Feb 13, 2025
@minrk
Copy link
Member

minrk commented Feb 13, 2025

Looking at branch protection rules for #1318, I think the rules were prohibiting pre-commit.ci from pushing to its branch. I think I fixed it by excluding pre-commit-ci-update-config and dependabot brnaches, but hard to say how to tell, since there's no way to force a PR.

@Carreau
Copy link
Member Author

Carreau commented Feb 13, 2025

Then I suggest we at least merge this to have an update next monday instead of waiting a month.

.pre-commit-config.yaml Outdated Show resolved Hide resolved
@minrk minrk merged commit 4f42f03 into ipython:main Feb 13, 2025
27 of 32 checks passed
@Carreau
Copy link
Member Author

Carreau commented Feb 13, 2025

thanks !

@Carreau Carreau deleted the pcu branch February 13, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants