Skip to content

precommit warns on main + instructions for fix #26170

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

Closed
wants to merge 1 commit into from
Closed

Conversation

story645
Copy link
Member

@story645 story645 commented Jun 22, 2023

Added a branch project rule to the pre-commit so that it complains if you try to commit directly to main. I figure folks who really need to push to main also know how to work around it. based on convo in call w/ @greglucas

Not quite sure this is the best way to handle the error message but figure we need to give folks a fix.
Um sorry, did not mean to open this branch on the mpl repo. (Also don't know how I did that since I branched from my local fork 😦)

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