Skip to content

Commit

Permalink
ci/test: re-enable unused deps lint
Browse files Browse the repository at this point in the history
But mark it to soft fail. I think the failures we were seeing before
might be an upstream bug for which there is no known workaround.

See: est31/cargo-udeps#103.
  • Loading branch information
benesch authored and Andi Wang committed Apr 11, 2022
1 parent 4dda441 commit 8a79a3b
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions ci/test/pipeline.template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,15 +68,16 @@ steps:
agents:
queue: mac

# TODO(guswynn): investigate flakiness of reading files
# - id: unused-deps
# label: Unused dependencies
# command: bin/ci-builder run nightly bin/unused-deps
# inputs:
# - Cargo.lock
# - "**/Cargo.toml"
# - "**/*.rs"
# timeout_in_minutes: 30
- id: unused-deps
label: Unused dependencies
command: bin/ci-builder run nightly bin/unused-deps
inputs:
- Cargo.lock
- "**/Cargo.toml"
- "**/*.rs"
timeout_in_minutes: 30
# https://github.com/est31/cargo-udeps/issues/103
soft_fail: true

- id: lint-docs
label: Lint docs
Expand Down

0 comments on commit 8a79a3b

Please sign in to comment.