Skip to content

docs: fix Rust version 1.93.0 → 1.96.1 in all 6 READMEs (Fixes #5781) - #5789

Open
yunaremaia wants to merge 1 commit into
tinyhumansai:mainfrom
yunaremaia:fix/docs-rust-version-1.96.1
Open

docs: fix Rust version 1.93.0 → 1.96.1 in all 6 READMEs (Fixes #5781)#5789
yunaremaia wants to merge 1 commit into
tinyhumansai:mainfrom
yunaremaia:fix/docs-rust-version-1.96.1

Conversation

@yunaremaia

@yunaremaia yunaremaia commented Aug 26, 2026

Copy link
Copy Markdown

Fixes #5781

Problem

rust-toolchain.toml pins channel = "1.96.1" because rusqlite 0.40 / libsqlite3-sys 0.38 require cfg_select! (stabilized in 1.96, rust-lang/rust#115585). CONTRIBUTING.md already says 1.96.1, but all 6 READMEs (README.md + 5 translations) still say 1.93.0. Following the README causes build failures.

Solution

Update all 6 files to 1.96.1:

  • README.md
  • docs/README.zh-CN.md
  • docs/README.de.md
  • docs/README.ja-JP.md
  • docs/README.ko.md
  • docs/README.ur-pk.md

Verified: grep -rn "1.96" README.md docs/README*.md shows 6/6 updated, matches rust-toolchain.toml and CONTRIBUTING.md.

Note: CONTRIBUTING-BEGINNERS.md and Dockerfile also reference 1.93.0 — left out of scope per issue acceptance criteria, happy to follow up if desired.

Summary by CodeRabbit

  • Documentation
    • Updated contributor setup requirements to specify Rust 1.96.1 as the minimum version.
    • Synchronized the requirement across English, German, Japanese, Korean, Urdu, and Simplified Chinese documentation.

…mansai#5781)

README and all 5 translations instructed contributors to install Rust 1.93.0, but rust-toolchain.toml pins 1.96.1 because rusqlite 0.40 / libsqlite3-sys 0.38 require cfg_select! (stabilized in 1.96, rust-lang/rust#115585). Following the README's instructions with 1.93.0 causes build failures. Update all 6 README files to 1.96.1 to match rust-toolchain.toml and CONTRIBUTING.md.

Signed-off-by: Yunare Maia <yunare@gmail.com>
@yunaremaia
yunaremaia requested a review from a team August 26, 2026 15:39

@tinysweeper tinysweeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out

@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Aug 26, 2026
@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: aef391d3-9c84-4968-aa9c-fa94dbe61f89

📥 Commits

Reviewing files that changed from the base of the PR and between 77fddf5 and ef68c87.

📒 Files selected for processing (6)
  • README.md
  • docs/README.de.md
  • docs/README.ja-JP.md
  • docs/README.ko.md
  • docs/README.ur-pk.md
  • docs/README.zh-CN.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The six English and translated README files now require Rust 1.96.1 instead of Rust 1.93.0 in contributor setup instructions. Other required tools remain unchanged.

Changes

Contributor prerequisite documentation

Layer / File(s) Summary
Update documented Rust version
README.md, docs/README.de.md, docs/README.ja-JP.md, docs/README.ko.md, docs/README.ur-pk.md, docs/README.zh-CN.md
All six README files now specify Rust 1.96.1 for source contributions. The required rustfmt and clippy components remain unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to ef68c

This updates documentation to match the repository’s configured Rust version without changing product or runtime behavior; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: senamakel

Poem

A rabbit checks the Rusty guide,

“One point nine six!” it says with pride.
Six README paths now match the pin,
No old version hides within.
rustfmt and clippy still abide.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the Rust version update across all six README files and references issue #5781.
Linked Issues check ✅ Passed The PR updates all six README files from Rust 1.93.0 to 1.96.1, satisfying the requirements in issue #5781 and aligning the documented prerequisite with the pinned toolchain.
Out of Scope Changes check ✅ Passed The changes are limited to the six README prerequisite updates required by issue #5781. No unrelated changes are present.
Docstring Coverage ✅ Passed 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…
Full details: Docstring Coverage

Explanation

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.)

Warning

Your free Security trial is over. An organization admin can activate billing to continue.


Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ef68c87d43

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
New contributor? Start with [`CONTRIBUTING.md`](./CONTRIBUTING.md) for the fork/PR workflow and local validation commands, or use the copy-paste AI-agent prompt in [`CONTRIBUTING-BEGINNERS.md`](./CONTRIBUTING-BEGINNERS.md#optional--let-an-ai-coding-agent-guide-you). The short path is:

1. Install Git, Node.js 24+, pnpm 10.10.0, Rust 1.93.0 (`rustfmt` + `clippy`), CMake, Ninja, ripgrep, and the platform desktop build prerequisites.
1. Install Git, Node.js 24+, pnpm 10.10.0, Rust 1.96.1 (`rustfmt` + `clippy`), CMake, Ninja, ripgrep, and the platform desktop build prerequisites.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update linked setup docs to the same Rust pin

When readers follow the “Getting Set Up” link from this same section, gitbooks/developing/getting-set-up.md still tells them to install Rust 1.93.0 (:22, :31-32, :44-45), and gitbooks/developing/building-rust-core.md still says the pinned channel is 1.93.0. Because those public contributor docs remain one click from the updated README and contradict rust-toolchain.toml, contributors following the longer setup path can still end up with the toolchain this change is trying to avoid; include the linked setup docs in this bump.

AGENTS.md reference: AGENTS.md:L16-L16

Useful? React with 👍 / 👎.

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

Labels

priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README says Rust 1.93.0 but rust-toolchain.toml requires 1.96.1

1 participant