Skip to content

Commit

Permalink
remove cargo-deny check from workflow since it is completly broken fo…
Browse files Browse the repository at this point in the history
…r now
  • Loading branch information
vemonet committed Sep 19, 2024
1 parent 4ffafda commit 2366706
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 7 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,9 @@ jobs:
- run: rustup update && cargo install cargo-deny
- run: cargo fmt -- --check
- run: cargo clippy --all --all-targets --all-features
- run: cargo deny check
# The devs behind cargo-deny seems completely lost with basic config setup...
# Probably better to completely get rid of it tbh https://github.com/EmbarkStudios/cargo-deny/issues/696
# - run: cargo deny check

test-rust:
name: 🧪 Test Rust
Expand Down
10 changes: 5 additions & 5 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@ repos:
entry: cargo fmt
language: system
pass_filenames: false
- id: deny
name: ❌ Check dependencies
entry: cargo deny check licenses
language: system
pass_filenames: false
# - id: deny
# name: ❌ Check dependencies
# entry: cargo deny check licenses
# language: system
# pass_filenames: false
ci:
autofix_commit_msg: 🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
autoupdate_commit_msg: ⬆ [pre-commit.ci] pre-commit autoupdate
2 changes: 1 addition & 1 deletion scripts/install-dev.sh
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@ fi
rustup update
rustup toolchain install nightly # For tarpaulin

cargo install wasm-pack cargo-tarpaulin cargo-deny cargo-outdated git-cliff
cargo install --force wasm-pack cargo-tarpaulin cargo-deny cargo-outdated git-cliff

pre-commit install

0 comments on commit 2366706

Please sign in to comment.