quiche icon indicating copy to clipboard operation
quiche copied to clipboard

ci: add cargo-machete in the workflow

Open junhochoi opened this issue 1 year ago • 1 comments

Run cargo-machete for unused dependency check, only on a default ssl target. Also removed unused dependencies recommended by an initial run of cargo-machete.

junhochoi avatar Jul 12 '24 02:07 junhochoi

Nice! Why only run on matrix.tls-feature == ''

Because it seems redundant on other tls features. Also installing cargo-machete takes some time. Instead we can try cargo-quickinstall to save an installation time, but it will produce the same result regardless of tls-feature anyway.

junhochoi avatar Jul 12 '24 03:07 junhochoi

Merged, thanks @junhochoi! (and sorry for the delay)

ghedo avatar Jan 13 '25 17:01 ghedo