← Back to context

Comment by schindlabua

9 hours ago

This might be a me problem but I extensively manipulate the git history all the time which makes me loathe git hooks. A commit should take milliseconds, not a minute.

it’s not just you.

i regularly edit history of PRs for a variety of reasons and avoid pre-commit when possible.

put it all in CI thank you please — gimme a big red X on my pipeline publicly telling me i’ve forgotten to do something considered important.

You do seem to be doing it wrong. Extensive manipulation of the record and slow hooks are both undesirable.

  • I would reckon cleaning up your branch before opening a pull request is good practice. I also rebase a lot, aswell as git reset, and I use wip commits.

    Slow hooks are also not a problem in projects I manage as I don't use them.

    • No, I would not and don't do that. It is better to leave the PR commits separate and atomic so reviewers can digest them more easily. You just squash on merge.

      > Slow hooks are also not a problem in projects I manage as I don't use them.

      You bypass the slow hooks you mentioned? Why even have hooks then?

      5 replies →