Skip to content

CI: enforce clippy -D warnings #162

@1nonlypiece

Description

@1nonlypiece

Description

Deny warnings in contracts crate.

Requirements and context

  • Efficient review

  • Clear rollback / no production impact for CI-only tasks

  • Must be secure, fully tested, and documented

  • Should be efficient and easy to review for auditors and integrators

Suggested execution

  1. Fork the repository and create a branch: git checkout -b ci/clippy
  2. Implement the changes listed below
  3. Open a draft PR early for design feedback

Implement

  • Touch .github/workflows/*.yml or tarpaulin.toml as needed
  • Document in README.md how to run locally

Tests and documentation

  • Add or extend automated tests: cargo test, CI logs
  • Add or update documentation: README.md
  • Add Rustdoc (///) on public contract APIs where applicable (NatSpec-style)
  • Validate stated security assumptions explicitly in PR description

Test and commit

  1. Run the full contract test suite (e.g. cargo test, Soroban local sandbox if applicable)
  2. Cover edge cases called out in this issue
  3. Attach test output summary and short security notes in the PR
  4. Ensure minimum 95% line coverage for touched modules (cargo tarpaulin / project standard)

Example commit message

chore(ci): clippy

Guidelines

Item Target
Test coverage 95% on changed code
Documentation Clear, versioned, matches on-chain behavior
Timeframe 96 hours from assignment

Metadata

Metadata

Assignees

Labels

Stellar WaveIssues in the Stellar wave programci-cdCI/CDsmart-contractSoroban / on-chain work

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions