Run Rust formatting, tests, and Clippy in CI
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 45/100
调研方向
先从 .github/workflows/ci.yml:27-44 和 46-62 中的两个验证作业开始,然后检查 ./scripts/test.py --operation validate 和 src/lib.rs。运行列出的 cargo fmt、cargo test 和 cargo clippy 命令以复现当前失败;完成的标准是 CI 强制执行这些检查,生成的 src/roc_platform_abi.rs 具有范围狭窄且有文档说明的例外,并且所要求的 host-helper 测试和贡献者说明都已存在。
由索引模型根据 Issue 内容生成。
描述
Context
The CI jobs named “Format, check, and test examples” run ./scripts/test.py --operation validate:
.github/workflows/ci.yml:27-44.github/workflows/ci.yml:46-62
That validation runs Roc tests/format/check operations, but it does not run Rust formatting, Rust tests, or Clippy. At commit 295000a during the release-candidate audit:
cargo check --locked --all-targetspassed;cargo fmt --all -- --checkfailed on formatting insrc/lib.rs;cargo test --locked --all-targetspassed but ran zero Rust tests;cargo clippy --locked --all-targets -- -D warningsfailed on generated acronym names and a raw-pointer ABI lint.
The generated src/roc_platform_abi.rs needs targeted lint allowances rather than disabling useful linting for handwritten host code.
Suggested direction
Add a dedicated Rust validation job or extend the current validation job to run:
cargo fmt --all -- --checkcargo test --locked --all-targetscargo clippy --locked --all-targets -- -D warnings
Configure narrow allowances for generated glue. Add unit tests for pure host helpers and error mappings so cargo test provides meaningful coverage in addition to end-to-end Roc examples.
Acceptance criteria
- Rust formatting is required by CI and the current formatting difference is fixed.
- Handwritten Rust passes Clippy with warnings denied.
- Generated glue has documented, narrowly scoped lint allowances.
- Rust tests cover representative HTTP method/error mapping, TCP read helpers, SQLite lifecycle helpers, and filesystem classification.
- Contributor verification instructions list the same commands CI runs.
- 主要语言
- Rust
- 星标
- 121
- 派生
- 46
- 平均合并
- 20 小时 53 分钟
- 30 天内合并 PR
- 11
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
roc-lang/basic-cli 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 45/100
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 25/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 45/100
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 52/100
维护者通常 1 天内回复
查看 roc-lang/basic-cli 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
bug good first issue
难度 2/5 1-3 小时 新手友好度 84/100
repowise-dev/repowise#3374 ·
维护者通常 1 天内回复
-
awaiting-response bug needs-triage
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 3 天内回复
-
难度 2/5 1-3 小时 新手友好度 75/100
objectionary/sodg.rs#301 ·
-
难度 2/5 1-3 小时 新手友好度 62/100
HakanSeven12/OpenCADStudio#1706 · 1 条评论 ·
维护者通常 1 天内回复