feat: Add cargo machete (#448)

* adding cargo machete to check unused dependencies

* remove default values

* adding a tag version instead of main

* running cargo machete natively

* removing unused dependency urlencoding

* bug fixes

Author: vaibhawvipul

.github/workflows/ci.yml

@@ -50,6 +50,9 @@ jobs:
       - name: Cargo sort
         run: make cargo-sort
 
+      - name: Cargo Machete
+        run: make cargo-machete
+
   build:
     runs-on: ${{ matrix.os }}
     strategy:

Cargo.toml

@@ -84,7 +84,6 @@ tempfile = "3.8"
 tokio = { version = "1", default-features = false }
 typed-builder = "^0.18"
 url = "2"
-urlencoding = "2"
 uuid = "1.6.1"
 volo-thrift = "0.10"
 hive_metastore = "0.1.0"

Makefile

@@ -32,6 +32,10 @@ cargo-sort:
 	cargo install cargo-sort
 	cargo sort -c -w
 
+cargo-machete:
+	cargo install cargo-machete
+	cargo machete
+
 fix-toml:
 	cargo install taplo-cli --locked
 	taplo fmt

crates/catalog/rest/Cargo.toml

@@ -42,7 +42,6 @@ serde_derive = { workspace = true }
 serde_json = { workspace = true }
 tokio = { workspace = true, features = ["sync"] }
 typed-builder = { workspace = true }
-urlencoding = { workspace = true }
 uuid = { workspace = true, features = ["v4"] }
 
 [dev-dependencies]