Skip to content

docs: fix minimum hardware requirement wording - #7587

Open
fmterrors wants to merge 4 commits into
ChainSafe:mainfrom
fmterrors:feature_fmterrors
Open

docs: fix minimum hardware requirement wording#7587
fmterrors wants to merge 4 commits into
ChainSafe:mainfrom
fmterrors:feature_fmterrors

Conversation

@fmterrors

@fmterrors fmterrors commented Sep 3, 2026

Copy link
Copy Markdown

Summary of changes

Changes introduced in this pull request:

  • Correct minium to minimum in the hardware requirements introduction.
  • Leave all hardware recommendations and numerical requirements unchanged.

Reference issue to close (if applicable)

Closes #ISSUE_NUMBER

Other information and links

AI assistance disclosure: No

Change checklist

  • I have performed a self-review of my own code — check after your own final review,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (not applicable to a spelling-only documentation change),
  • I have made sure the CHANGELOG is up-to-date (not applicable; no user-visible behavior changed).

Outside contributions

  • This pull request is based on an issue that a maintainer has accepted (see #ISSUE_NUMBER) — check only after acceptance.
  • I have read and agree to the CONTRIBUTING document — read it yourself before checking.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request — read it yourself before checking.

Summary by CodeRabbit

  • Documentation
    • Corrected a spelling error in the introductory paragraph of the hardware requirements guide.

Signed-off-by: fmterrors <fmterrors@outlook.com>
@fmterrors
fmterrors requested a review from a team as a code owner September 3, 2026 06:50
@fmterrors
fmterrors requested review from LesnyRumcajs and akaladarshi and removed request for a team September 3, 2026 06:50
@CLAassistant

CLAassistant commented Sep 3, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Sep 3, 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: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 26e440f0-42fc-4614-b332-1cd2c1fca153

📥 Commits

Reviewing files that changed from the base of the PR and between 57cde54 and 9631177.

📒 Files selected for processing (1)
  • docs/docs/users/getting_started/hardware-reqs.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • filecoin-project/lotus (manual)

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


Walkthrough

The hardware requirements page corrects the misspelling “minium” to “minimum” in its introductory paragraph.

Changes

Hardware Requirements Documentation

Layer / File(s) Summary
Correct hardware requirements text
docs/docs/users/getting_started/hardware-reqs.md
The introductory paragraph now uses the correct spelling “minimum”.

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

Merge Risk: ⚪ Minimal · up to 60550

This change corrects a spelling error in the hardware requirements documentation without changing any requirements or recommendations, and is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the documentation wording correction from “minium” to “minimum.”
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. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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

@akaladarshi

Copy link
Copy Markdown
Contributor

@fmterrors The commits need to be signed before we can merge this.

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.

5 participants