feat: add `pre commit` hooks
Closes #11 (closed).
Merge request reports
Activity
Filter activity
assigned to @Tomer.Shterenberg
- Resolved by Sebastian Birunt
I hit exactly the same bug in remote execution in bazel/toolchain_utils!37. Solving that bug will allow rollout to our projects.
We need to reproduce the remote execution bug on the bazel/pre-commit> project to prevent regressions.
- Resolved by Tomer Shterenberg
Can we add the de-facto
.git-blame-ignore-revsto ignore pure linting/formatting commits ingit blame?git@2.23learnt how to do this with the following setup:$ git config --global blame.ignoreRevsFile .git-blame-ignore-revs $ git config --global blame.markIgnoredLines true $ git config --global blame.markUnblamableLines trueYou can add the commit SHA for the commit that runs the hooks across all files.
- Resolved by Tomer Shterenberg
- Resolved by Tomer Shterenberg
- Resolved by Tomer Shterenberg
- Resolved by Tomer Shterenberg
mentioned in issue pre-commit#6 (closed)
added 10 commits
-
8230bab5...e2ceb10d - 4 commits from branch
main - a3f450d7 - ci: update ruleset to v1.5.0
- 1fbe71fc - feat: add 'pre-commit' hooks
- 215bb5f8 - chore: run 'pre-commit' hooks
- 0bee1fd7 - chore: add .git-blame-ignore-revs
- 551c2f69 - chore: updategrade to Bazel `8.2.1`
- d487e045 - docs(contributing): add hooks installation
Toggle commit list-
8230bab5...e2ceb10d - 4 commits from branch