From b3567a394f54a220cf01a92e16f90e5554a4919b Mon Sep 17 00:00:00 2001 From: tottoto Date: Wed, 8 Feb 2023 23:08:21 +0900 Subject: [PATCH] chore: Add cargo-machete to detect unused dependencies (#1259) --- .github/workflows/CI.yml | 6 ++++++ examples/Cargo.toml | 3 +++ interop/Cargo.toml | 3 +++ tests/ambiguous_methods/Cargo.toml | 3 +++ tests/compression/Cargo.toml | 3 +++ tests/disable_comments/Cargo.toml | 3 +++ tests/extern_path/my_application/Cargo.toml | 3 +++ tests/extern_path/uuid/Cargo.toml | 3 +++ tests/included_service/Cargo.toml | 3 +++ tests/integration_tests/Cargo.toml | 3 +++ tests/same_name/Cargo.toml | 3 +++ tests/service_named_result/Cargo.toml | 3 +++ tests/service_named_service/Cargo.toml | 3 +++ tests/stream_conflict/Cargo.toml | 3 +++ tests/wellknown-compiled/Cargo.toml | 3 +++ tests/wellknown/Cargo.toml | 3 +++ 16 files changed, 51 insertions(+) diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml index 0b293da..1c159de 100644 --- a/.github/workflows/CI.yml +++ b/.github/workflows/CI.yml @@ -27,6 +27,10 @@ jobs: uses: baptiste0928/cargo-install@v1 with: crate: cargo-hack + - name: Install cargo-machete + uses: baptiste0928/cargo-install@v1 + with: + crate: cargo-machete - name: Install Protoc uses: arduino/setup-protoc@v1 with: @@ -34,6 +38,8 @@ jobs: - uses: Swatinem/rust-cache@v2 - name: Check fmt run: cargo fmt -- --check + - name: Check unused dependencies + run: cargo machete - name: Check features run: cargo hack check --all --ignore-private --each-feature --no-dev-deps - name: Check all targets diff --git a/examples/Cargo.toml b/examples/Cargo.toml index 6c776d5..0e37b42 100644 --- a/examples/Cargo.toml +++ b/examples/Cargo.toml @@ -293,3 +293,6 @@ tower-http = { version = "0.3", optional = true } [build-dependencies] tonic-build = { path = "../tonic-build", features = ["prost"] } + +[package.metadata.cargo-machete] +ignored = ["prost-types"] diff --git a/interop/Cargo.toml b/interop/Cargo.toml index bc975e6..f7b4c42 100644 --- a/interop/Cargo.toml +++ b/interop/Cargo.toml @@ -35,3 +35,6 @@ tracing-subscriber = {version = "0.3", features = ["env-filter"]} [build-dependencies] tonic-build = {path = "../tonic-build", features = ["prost"]} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/ambiguous_methods/Cargo.toml b/tests/ambiguous_methods/Cargo.toml index d0d193a..03f4c55 100644 --- a/tests/ambiguous_methods/Cargo.toml +++ b/tests/ambiguous_methods/Cargo.toml @@ -14,3 +14,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/compression/Cargo.toml b/tests/compression/Cargo.toml index 5d1010b..43e64b1 100644 --- a/tests/compression/Cargo.toml +++ b/tests/compression/Cargo.toml @@ -21,3 +21,6 @@ tower-http = {version = "0.3", features = ["map-response-body", "map-request-bod [build-dependencies] tonic-build = {path = "../../tonic-build" } + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/disable_comments/Cargo.toml b/tests/disable_comments/Cargo.toml index c694fc0..9490687 100644 --- a/tests/disable_comments/Cargo.toml +++ b/tests/disable_comments/Cargo.toml @@ -15,3 +15,6 @@ tonic = { path = "../../tonic" } [build-dependencies] prost-build = "0.11.6" tonic-build = { path = "../../tonic-build" } + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/extern_path/my_application/Cargo.toml b/tests/extern_path/my_application/Cargo.toml index a2eeb6b..d30f62c 100644 --- a/tests/extern_path/my_application/Cargo.toml +++ b/tests/extern_path/my_application/Cargo.toml @@ -15,3 +15,6 @@ uuid = {package = "uuid1", path = "../uuid"} [build-dependencies] tonic-build = {path = "../../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/extern_path/uuid/Cargo.toml b/tests/extern_path/uuid/Cargo.toml index df1e497..694e080 100644 --- a/tests/extern_path/uuid/Cargo.toml +++ b/tests/extern_path/uuid/Cargo.toml @@ -12,3 +12,6 @@ version = "0.1.0" prost = "0.11" [build-dependencies] prost-build = "0.11.6" + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/included_service/Cargo.toml b/tests/included_service/Cargo.toml index 77469fa..b7e4d72 100644 --- a/tests/included_service/Cargo.toml +++ b/tests/included_service/Cargo.toml @@ -14,3 +14,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/integration_tests/Cargo.toml b/tests/integration_tests/Cargo.toml index e52d1c6..d9c20b5 100644 --- a/tests/integration_tests/Cargo.toml +++ b/tests/integration_tests/Cargo.toml @@ -29,3 +29,6 @@ tracing-subscriber = {version = "0.3", features = ["env-filter"]} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/same_name/Cargo.toml b/tests/same_name/Cargo.toml index e2c69a7..b336c28 100644 --- a/tests/same_name/Cargo.toml +++ b/tests/same_name/Cargo.toml @@ -14,3 +14,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/service_named_result/Cargo.toml b/tests/service_named_result/Cargo.toml index 1730235..f254bae 100644 --- a/tests/service_named_result/Cargo.toml +++ b/tests/service_named_result/Cargo.toml @@ -12,3 +12,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/service_named_service/Cargo.toml b/tests/service_named_service/Cargo.toml index be97922..3bb4e5c 100644 --- a/tests/service_named_service/Cargo.toml +++ b/tests/service_named_service/Cargo.toml @@ -14,3 +14,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/stream_conflict/Cargo.toml b/tests/stream_conflict/Cargo.toml index 866cba9..f651409 100644 --- a/tests/stream_conflict/Cargo.toml +++ b/tests/stream_conflict/Cargo.toml @@ -14,3 +14,6 @@ tonic = { path = "../../tonic" } [build-dependencies] tonic-build = { path = "../../tonic-build" } + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/wellknown-compiled/Cargo.toml b/tests/wellknown-compiled/Cargo.toml index 60842b4..4fb6726 100644 --- a/tests/wellknown-compiled/Cargo.toml +++ b/tests/wellknown-compiled/Cargo.toml @@ -18,3 +18,6 @@ tonic = {path = "../../tonic"} [build-dependencies] prost-build = "0.11.6" tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost"] diff --git a/tests/wellknown/Cargo.toml b/tests/wellknown/Cargo.toml index 194316e..efe0f4c 100644 --- a/tests/wellknown/Cargo.toml +++ b/tests/wellknown/Cargo.toml @@ -15,3 +15,6 @@ tonic = {path = "../../tonic"} [build-dependencies] tonic-build = {path = "../../tonic-build"} + +[package.metadata.cargo-machete] +ignored = ["prost", "prost-types"]