← Back to issue list

docs: spell out pull requests

View original Github issue

Metadata

Project
craft-grammar
Number
#221
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #221 to fix Vale spellcheck error by spelling out PRs as pull requests in docs/explanation/documentation.rst.

Resolution: closed resolved

Reason: Resolved by merging PR #221, which updates the documentation to spell out pull requests to satisfy Vale's spellcheck rules.

Evaluated by: mr-cal-bot

No scores available.

Issue body

## Summary Vale's `Canonical.000-US-spellcheck` flags the abbreviation `PRs` in `docs/explanation/documentation.rst`: ``` explanation/documentation.rst 32:19 error The word 'PRs' seems to be misspelled. Canonical.000-US-spellcheck ``` `docs-lint` runs the Vale spelling check, but on `main` the "Run documentation linters" CI step is skipped by the paths filter because main commits only ever touch docs files. Any PR that touches non-docs files enables that step, which is how this surfaced. The fix spells the term out ("pull requests"), which also matches the wording starbase uses for the same sentence. Pre-existing issue, unrelated to the Starbase sync; split out so the merge diff stays reviewable. ## Test plan - [x] `make -C docs spelling` reports 0 errors - [x] `make docs` builds successfully

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — closed resolved Merged PR #221 to fix Vale spellcheck error by spelling out PRs as pull requests in docs/explanation/documentation.rst.

Update history

Date Change
merged
created

Related issues

Issue Project State Summary Similarity
#205 docs: reword "PRs" to "pull requests" for Vale spellcheck debcraft merged Merged PR #205 to fix Vale spellcheck failure by rewording "PRs" to "pull requests" in docs/explanation/documentation.rst.
90%
#445 docs: resolve Vale spellcheck errors from vale 3.22 craft-store merged Merged into main to resolve Vale 3.22.0 spellcheck failures in docs by replacing "PRs" with "pull requests" and adding "venv" and "mv" to the custom wordlist.
78%