Skip to content

fix: spellcheck issues and pin cargo spellcheck#6719

Merged
LesnyRumcajs merged 1 commit intomainfrom
fix-spellcheck
Mar 11, 2026
Merged

fix: spellcheck issues and pin cargo spellcheck#6719
LesnyRumcajs merged 1 commit intomainfrom
fix-spellcheck

Conversation

@LesnyRumcajs
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs commented Mar 11, 2026

Summary of changes

Changes introduced in this pull request:

  • new cargo-spellcheck got released and it's more picky; it's good but we don't want it to fail all of a sudden so I pinned the version

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • 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 (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • I have read and agree to the CONTRIBUTING document.
  • 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.

Summary by CodeRabbit

Release Notes

  • Documentation

    • Improved contributor guidelines with consistent formatting, spacing, and header styling throughout the documentation.
  • Chores

    • Added new entries to development dictionary for improved tool recognition.
    • Pinned build tool versions for consistent development environment setup across all configuration files.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 11, 2026

Walkthrough

Updates documentation tooling configuration and dependencies by pinning cargo-spellcheck to version 0.15.7 in workflow and task files, adding new dictionary entries to the spell check dictionary, and applying minor formatting corrections to the contributing guide.

Changes

Cohort / File(s) Summary
Spell Check Dictionary
.config/forest.dic
Added 4 new entries (nextest, SDK, toml, traceback) and updated dictionary size counter from 258 to 262.
CI/Build Tooling Configuration
.github/workflows/docs-check.yml, mise.toml
Pinned cargo-spellcheck dependency to version 0.15.7 in both the GitHub Actions workflow install step and the mise task runner configuration.
Documentation Formatting
CONTRIBUTING.md
Applied cosmetic formatting corrections including emoji spacing, header capitalization, and punctuation standardization (e.g., "Styleguides" → "Style Guides", "🤖Code formatting" → "🤖 Code formatting").

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • fix: docs linting #6452: Modifies the same docs-check workflow and mise.toml CI configuration files related to documentation tooling.

Suggested labels

dependencies

Suggested reviewers

  • hanabi1224
  • akaladarshi
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: fixing spellcheck issues (dictionary updates in forest.dic and formatting in CONTRIBUTING.md) and pinning cargo-spellcheck to version 0.15.7.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-spellcheck

Warning

Review ran into problems

🔥 Problems

Linked repositories: Couldn't analyze filecoin-project/lotus - clone failed: Clone operation failed: Cloning into '/home/jailuser/git'...
warning: templates not found in /usr/share/git-core/templates
fatal: unable to access 'https://github.com/filecoin-project/lotus.git/': Failed to connect to github.com port 443 after 135204 ms: Couldn't connect to server


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

@LesnyRumcajs LesnyRumcajs marked this pull request as ready for review March 11, 2026 10:02
@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner March 11, 2026 10:02
@LesnyRumcajs LesnyRumcajs requested review from akaladarshi and hanabi1224 and removed request for a team March 11, 2026 10:02
@LesnyRumcajs LesnyRumcajs enabled auto-merge March 11, 2026 10:25
@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue Mar 11, 2026
Merged via the queue into main with commit 0214cd5 Mar 11, 2026
50 checks passed
@LesnyRumcajs LesnyRumcajs deleted the fix-spellcheck branch March 11, 2026 11:02
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.

2 participants