Skip to content

Resolves #5741: Fix typos in comments, docs and spec descriptions - #5742

Merged
dorner merged 1 commit into
rubyforgood:mainfrom
geographybuff:5741-fix-typos
Oct 12, 2026
Merged

dorner merged 1 commit into
rubyforgood:mainfrom
geographybuff:5741-fix-typos

Conversation

@geographybuff

Copy link
Copy Markdown
Contributor

Resolves #5741

Description

This PR fixes 20 typos in comments, docs and spec descriptions. It changes spelling only, no behavior.

  • Comments: Gemfile, _variables.scss, distributions_controller.rb, two Stimulus controllers, distribution_itemized_breakdown_service.rb, db/seeds.rb
  • Docs: ADRs 0008 and 0009, the partners getting-started guide, the inventory-audits image alt text, and the application overview in the wiki docs
  • Spec descriptions: describe "associations" in the tag and tagging specs, and "attempting" in four system spec names, plus one spec comment

Deliberately not changed

  • Identifiers and variant spellings: cancelation (routes, controller, views), Requestor (also a CSV export header), the $diaper-notifcation-* SCSS variables (only the comment above them was fixed), and Devise's Rememberable
  • Data and fixtures: db/base_items.json ("Childrens"), the spec fixture CSV ("Familes"), and the county list
  • Deliberate text (lorem ipsum, "OH NOES", which a spec asserts) and vendored or generated assets (AdminLTE, vendor/, public/)
  • config/locales/en.yml, which had no hits, so no i18n keys or values change

AI disclosure: prepared with Claude Code (Claude Opus 5.5). It ran a codespell sweep, checked each candidate in context, and made the edits.

Type of change

  • Documentation update

How Has This Been Tested?

  • ruby -c passes on all 11 changed Ruby files (including the Gemfile), and node --check passes on the 2 changed JS files.
  • The spec changes only touch describe/it strings and one comment. The full suite (bundle exec rake) was not run locally; CI will run it.

🤖 Generated with Claude Code

Spelling-only changes; no identifiers, strings asserted on by specs,
locale keys, migrations or vendored files are touched.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@geographybuff
geographybuff marked this pull request as ready for review October 8, 2026 03:55

@dorner dorner left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I miss the individuality of the old comments 😂 Just kidding. Kind of.

@dorner
dorner merged commit 6453a3d into rubyforgood:main Oct 12, 2026
12 checks passed
@geographybuff

Copy link
Copy Markdown
Contributor Author

@dorner I can write boring black and white plain text comments with ASCII smileys without LLMs too if you want :)

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.

Fix typos in comments, docs and spec descriptions

2 participants