Problem
The project relies on make fmt and make test but lacks a dedicated linting step to catch Rust idiomatic errors, unused imports, or potential memory/performance issues.
Expected behavior
A lint target is added to the Makefile utilizing cargo clippy, and any existing warnings in the codebase are resolved.
Files to update
Makefile
src/lib.rs
src/vault.rs
src/strategy.rs
src/test.rs
Project relevance
Integrating Clippy ensures the Soroban smart contracts remain optimized, secure, and idiomatic as the open-source contributor base scales.
Acceptance criteria
Problem
The project relies on
make fmtandmake testbut lacks a dedicated linting step to catch Rust idiomatic errors, unused imports, or potential memory/performance issues.Expected behavior
A
linttarget is added to theMakefileutilizingcargo clippy, and any existing warnings in the codebase are resolved.Files to update
Makefilesrc/lib.rssrc/vault.rssrc/strategy.rssrc/test.rsProject relevance
Integrating Clippy ensures the Soroban smart contracts remain optimized, secure, and idiomatic as the open-source contributor base scales.
Acceptance criteria
Makefileincludeslint: cargo clippy --all-targets --all-features -- -D warnings