-
Notifications
You must be signed in to change notification settings - Fork 54
fix(docs): correct typos and improve consistency across articles #145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Added missing word "circles" in description of 'a' - Reworded all three distance descriptions for clarity and consistency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall good work @QuestSWE and thanks for the fixes.
There are some minor changes requested
- Avoid using contractions (such as "we're", use "we are" instead).
- One section and ordered list was changed to an unordered list. It should be ordered since the statements presented are done so in order of execution.
- A few places where
< >
got added around the xref links when it's not needed.
articles/tutorials/building_2d_games/22_snake_game_mechanics/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
articles/tutorials/building_2d_games/12_collision_detection/index.md
Outdated
Show resolved
Hide resolved
All the changes above have been resolved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SimonDarksideJ Should be good on this one
This pull request fixes multiple typographical, grammatical, and minor wording issues across various documentation files.
Additionally:
<param>
XML comment that referred to a non-existent parameterX = Y;
immediately overridden on the next line)These were small, safe cleanups to prevent confusion or dead code.
These edits follow the style and tone already established in the project.
Let me know if you’d like anything adjusted.