Skip to content

Conversation

@philipphofmann
Copy link
Member

The hook prevents committing directly to the main branch, but this can be annoying for developers who use a different Git workflow. If you accidentally try to commit to main, you have to create a new branch and rerun all pre-commit checks.

Since we already enforce branch protection rules for main via GitHub, this hook adds little value while reducing flexibility. Removing it allows for a more streamlined workflow without unnecessary friction.

#skip-changelog

The  hook prevents committing directly to the main branch, but this can
be annoying for developers who use a different Git workflow. If you
accidentally try to commit to main, you have to create a new branch and
rerun all pre-commit checks.

Since we already enforce branch protection rules for main via GitHub,
this hook adds little value while reducing flexibility. Removing it
allows for a more streamlined workflow without unnecessary friction.
Copy link
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

LGTM

@philipphofmann philipphofmann merged commit aea42ca into main Mar 14, 2025
8 checks passed
@philipphofmann philipphofmann deleted the chore/remove-pre-commit-no-commit branch March 14, 2025 11:38
@philipphofmann philipphofmann restored the chore/remove-pre-commit-no-commit branch April 29, 2025 13:37
@philipphofmann philipphofmann deleted the chore/remove-pre-commit-no-commit branch July 7, 2025 12:12
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