Skip to content

Fix typos, spelling errors, and factual issues in documentation - #32

Merged
mmmarinho merged 2 commits into
mainfrom
fix-typo-and-spelling-fixes
Aug 14, 2026
Merged

Fix typos, spelling errors, and factual issues in documentation#32
mmmarinho merged 2 commits into
mainfrom
fix-typo-and-spelling-fixes

Conversation

@mmmarinho

Copy link
Copy Markdown
Owner

Summary

This PR fixes 11 typos, spelling errors, and factual issues found across the documentation source files. These issues could confuse readers following the tutorial, especially the incorrect package names and message type references.

Changes

Spelling fixes

File Line Before After
docs/source/index.rst 70 librarys libraries
docs/source/installation.rst 51 derivates derivatives
docs/source/inspecting_topics.rst 286, 298 was though by was thought by
docs/source/transformations/index.rst 326 TransformedStamped TransformStamped

Grammar fixes

File Line Before After
docs/source/index.rst 22 The example code have The example code has
docs/source/index.rst 43 compatibility ... have been compatibility ... has been
docs/source/inspecting_services.rst 219 it's own its own
docs/source/transformations/tf2.rst 227 subscribers to not subscribers do not
docs/source/source_after_build.rst 123 resource re-source

Factual fixes (incorrect package name)

File Line Before After
docs/source/parameters_and_launch.rst 14 packages_with_interfaces package_with_interfaces
docs/source/publishers_and_subscribers.rst 59 packages_with_interfaces package_with_interfaces

Severity

  • High — Incorrect package names (packages_with_interfacespackage_with_interfaces) and message type (TransformedStampedTransformStamped) could mislead learners following the tutorial step by step.
  • Medium — Misspelled word "was though by" vs "was thought by" changes the meaning of the sentence.
  • Low — Grammar and spelling refinements for clarity.

This PR was created by an AI agent (OpenHands) on behalf of the user.

- index.rst: Fix subject-verb agreement ('code have' -> 'code has', 'compatibility have' -> 'compatibility has')
- index.rst: Fix misspelling 'librarys' -> 'libraries', adjust pronoun 'it' -> 'them'
- inspecting_topics.rst: Fix typo 'was though by' -> 'was thought by' (2 occurrences)
- installation.rst: Fix misspelling 'derivates' -> 'derivatives'
- parameters_and_launch.rst: Fix incorrect package name 'packages_with_interfaces' -> 'package_with_interfaces'
- publishers_and_subscribers.rst: Fix incorrect package name 'packages_with_interfaces' -> 'package_with_interfaces'
- transformations/index.rst: Fix incorrect message type 'TransformedStamped' -> 'TransformStamped'
- inspecting_services.rst: Fix grammar 'it's own' -> 'its own'
- source_after_build.rst: Fix ambiguous word 'resource' -> 're-source'
- transformations/tf2.rst: Fix grammar 'subscribers to not' -> 'subscribers do not'

@mmmarinho mmmarinho left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

👍

- Replace 26 instances of deprecated .. code :: with .. code-block ::
- Standardize sub-subsection underlines: convert ^ and ~ to + (22 lines across 6 files)
- Fix 9 section underline length mismatches
- Fix 2 broken .. note: directives (missing ::)
- Fix 1 extra space in ..  code-block:: directive
- Add missing trailing newlines to 13 files
- Remove trailing whitespace from all RST files
- Standardize :download: labels to use only the filename

@mmmarinho mmmarinho left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

👍

@mmmarinho
mmmarinho marked this pull request as ready for review August 14, 2026 22:16
@mmmarinho
mmmarinho merged commit 940e479 into main Aug 14, 2026
4 checks passed
@mmmarinho
mmmarinho deleted the fix-typo-and-spelling-fixes branch August 14, 2026 22:16
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