Skip to content

Commit

Permalink
ci: enable cargo machete check
Browse files Browse the repository at this point in the history
Signed-off-by: iGxnon <[email protected]>
  • Loading branch information
iGxnon committed Feb 4, 2024
1 parent 4d46aa3 commit 9082ab2
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,8 @@ jobs:
- name: Sort crates
run: cargo sort --check --workspace

# TODO: trigger `Build Env` to release `ghcr.io/xline-kv/build-env:latest` when this PR is merged, then uncomment the step
# - name: Check dependencies
# run: cargo machete --fix crates/
- name: Check dependencies
run: cargo machete crates/

- name: Workspace hack check
run: cargo hakari generate --diff && cargo hakari manage-deps --dry-run && cargo hakari verify
Expand Down
2 changes: 0 additions & 2 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,5 @@ members = [
]
resolver = "2"

# TODO: when cargo-machete support ignore workspace member crate, then add it into pre commit hook & CI
# ISSUE: https://github.com/bnjbvr/cargo-machete/issues/94
[workspace.metadata.cargo-machete]
ignored = ["prost", "workspace-hack"]
2 changes: 1 addition & 1 deletion crates/utils/src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1205,7 +1205,7 @@ mod tests {
server_cert_path = './cert.pem'
server_key_path = './key.pem'
client_ca_cert_path = './ca.pem'
[metrics]
enable = true
port = 9100
Expand Down

0 comments on commit 9082ab2

Please sign in to comment.