Skip to content

ci: 增加 rustfmt 和严格 clippy 质量门禁 - #91

Open
chxcode wants to merge 1 commit into
SmartDropLabs:mainfrom
chxcode:chxcode/ci-fmt-clippy-88
Open

ci: 增加 rustfmt 和严格 clippy 质量门禁#91
chxcode wants to merge 1 commit into
SmartDropLabs:mainfrom
chxcode:chxcode/ci-fmt-clippy-88

Conversation

@chxcode

@chxcode chxcode commented Jul 15, 2026

Copy link
Copy Markdown

变更说明

  • 在 GitHub Actions 的 Rust 工具链安装中启用 rustfmtclippy
  • 增加 cargo fmt --all -- --check 格式检查
  • 增加 cargo clippy --workspace --all-targets -- -D warnings 严格 lint 门禁
  • 修复启用严格 clippy 后暴露的现有轻量 lint
  • 对尚未迁移到 #[contractevent] 的现有事件调用进行局部 #[allow(deprecated)],避免全局放宽告警

验证

cargo fmt --all -- --check
cargo clippy --workspace --all-targets -- -D warnings
cargo test --workspace
cargo build --target wasm32v1-none --release

本地结果:

  • rustfmt 通过
  • 严格 clippy 通过,无全局 warning 豁免
  • Factory:29 个测试通过
  • Farming Pool:74 个测试通过
  • 合计 103 个测试通过
  • WASM release 构建通过

Closes #88

@netlify

netlify Bot commented Jul 15, 2026

Copy link
Copy Markdown

Deploy Preview for sdcontracts ready!

Name Link
🔨 Latest commit 26b9d5d
🔍 Latest deploy log https://app.netlify.com/projects/sdcontracts/deploys/6a576c0a7c5d7800084f2f85
😎 Deploy Preview https://deploy-preview-91--sdcontracts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@chxcode

chxcode commented Jul 15, 2026

Copy link
Copy Markdown
Author

PR 已提交并完成本地验证。补充说明:上游仓库的 GitHub Actions 对首次 fork PR 显示 action_required,尚未创建 job,需要维护者批准工作流运行;这不是测试失败。批准后新增的 rustfmt、严格 clippy、现有测试与 WASM build 步骤会在 CI 中执行。

PR: #91

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

infrastructure: CI never runs cargo clippy or cargo fmt --check, letting lint regressions and panic-safety fixes silently regress

1 participant