Skip to content

Commit

Permalink
Merge pull request #246 from akoshelev/deny-deny-warnings
Browse files Browse the repository at this point in the history
Remove deny(warnings)
  • Loading branch information
akoshelev authored Nov 17, 2022
2 parents 9802ce1 + f09e77d commit 5686d93
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ jobs:
name: Rust Checks
env:
CARGO_TERM_COLOR: always
RUSTFLAGS: -D warnings
RUSTDOCFLAGS: -D warnings

runs-on: ubuntu-latest

steps:
Expand Down
2 changes: 1 addition & 1 deletion pre-commit
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,6 @@ error() {

cargo build --benches --all-features || error "Benchmarks compilation errors"

cargo clippy --tests || error "Clippy errors"
cargo clippy --tests -- -D warnings || error "Clippy errors"

cargo test || error "Test failures"
2 changes: 1 addition & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#![deny(warnings, clippy::pedantic, clippy::clone_on_ref_ptr)]
#![deny(clippy::pedantic, clippy::clone_on_ref_ptr)]
// The following warnings are too noisy for us and having them enabled leads to polluting the
// code with allow annotations. Disabling them once per project here
#![allow(clippy::similar_names)]
Expand Down

0 comments on commit 5686d93

Please sign in to comment.