Skip to content

Commit

Permalink
build: remove pre-push hooks (#1115)
Browse files Browse the repository at this point in the history
  • Loading branch information
joshka committed May 19, 2024
1 parent 828d17a commit 4955380
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 24 deletions.
16 changes: 0 additions & 16 deletions .cargo-husky/hooks/pre-push

This file was deleted.

8 changes: 3 additions & 5 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,9 @@ documented.

### Run CI tests before pushing a PR

We're using [cargo-husky](https://github.com/rhysd/cargo-husky) to automatically run git hooks,
which will run `cargo make ci` before each push. To initialize the hook run `cargo test`. If
`cargo-make` is not installed, it will provide instructions to install it for you. This will ensure
that your code is formatted, compiles and passes all tests before you push. If you need to skip this
check, you can use `git push --no-verify`.
Running `cargo make ci` before pushing will perform the same checks that we do in the CI process.
It's not mandatory to do this before pushing, however it may save you time to do so instead of
waiting for GitHub to run the checks.

### Sign your commits

Expand Down
3 changes: 0 additions & 3 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,6 @@ unicode-width = "0.1"
anyhow = "1.0.71"
argh = "0.1.12"
better-panic = "0.3.0"
cargo-husky = { version = "1.5.0", default-features = false, features = [
"user-hooks",
] }
color-eyre = "0.6.2"
criterion = { version = "0.5.1", features = ["html_reports"] }
derive_builder = "0.20.0"
Expand Down

0 comments on commit 4955380

Please sign in to comment.