fix(docs): update README - #5784
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe contributor instructions in the English, German, Japanese, Korean, Urdu, and Chinese README files now require Rust 1.96.1 instead of Rust 1.93.0. ChangesRust prerequisite documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change corrects the Rust version shown to contributors without changing runtime behavior; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (6 skipped: 6 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
Comment |
Summary
rust-toolchain.tomlandCONTRIBUTING.md.Problem
rust-toolchain.toml, andCONTRIBUTING.mdalready documents 1.96.1.Solution
1.93.0to1.96.1in all six affected README files:README.mddocs/README.zh-CN.mddocs/README.de.mddocs/README.ja-JP.mddocs/README.ko.mddocs/README.ur-pk.mdrust-toolchain.tomlandCONTRIBUTING.md.Submission Checklist
Tests added or updated (happy path + at least one failure / edge case) per Testing Strategy
N/A: Documentation-only version correction; no runtime behavior changed.Diff coverage ≥ 80% — changed lines (Vitest + cargo-llvm-cov merged via
diff-cover) meet the gate enforced by.github/workflows/ci-lite.yml. Runpnpm test:coverageandpnpm test:rustlocally; PRs below 80% on changed lines will not merge.N/A: Documentation-only change; no executable code was modified.Coverage matrix updated — added/removed/renamed feature rows in
docs/TEST-COVERAGE-MATRIX.mdreflect this change (orN/A: behaviour-only change)N/A: No features, behavior, or coverage requirements changed.All affected feature IDs from the matrix are listed in the PR description under
## RelatedN/A: No affected feature IDs.No new external network dependencies introduced (mock backend used per Testing Strategy
N/A: No dependencies or network behavior changed.Manual smoke checklist updated if this touches release-cut surfaces (
docs/RELEASE-MANUAL-SMOKE.mdN/A: Documentation-only change; release-cut behavior is unaffected.Linked issue closed via
Closes #NNNin the## RelatedsectionCloses #ISSUE_NUMBERImpact
Related
Closes #5781
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
N/A: No Linear issue associated with this change.N/ACommit & Branch
fix-readme<8157f414602effbc09532bd5f096d6ab7ab9a52eValidation Run
pnpm --filter openhuman-app format:checkN/A: Documentation-only change; no application source formatting is affected.pnpm typecheckN/A: Documentation-only change; no TypeScript source changed.Focused tests:
N/A: Documentation-only change; no runtime behavior changed.Rust fmt/check (if changed):
N/A: No Rust source files changed.Tauri fmt/check (if changed):
N/A: No Tauri source files changed.Validation Blocked
command: N/Aerror: N/Aimpact: N/A: Validation was not blocked; the listed code-validation commands are not applicable to a documentation-only change.Behavior Changes
1.96.1instead of1.93.0when following the "Contributing from source" instructions.Parity Contract
N/A: No runtime guards, fallbacks, or dispatch logic changed.Duplicate / Superseded PR Handling
N/ASummary by CodeRabbit