Skip to content

[Docs] Various grammar and spelling fixes #107895

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

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

AThousandShips
Copy link
Member

Fixes to:

  • Possessive forms ("user" vs "user's")
  • Article ("a" vs "an")

For articles this enforces the general preference that file extensions are interpreted as being read as starting with "dot", so always using "a", as well as some case where English is confusing and consonant and vowel letters aren't pronounced as consonant and vowel sounds, for example "U" is often pronounced as "ju" (for example most abbreviations pronounce this as "ju"), and several consonants are pronounced with a leading "e" or "a" when in acronyms (for example "RID", pronounced as "aRaiDee"), also US English generally doesn't drop the "h" generally, but in abbreviations it is often pronounced "eitch" (for example HTTP)

As well as some straight up errors rather than confusion

@akien-mga akien-mga modified the milestones: 4.x, 4.5 Jun 23, 2025
Copy link
Member

@Mickeon Mickeon left a comment

Choose a reason for hiding this comment

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

I wonder how many of both this and godotengine/godot-docs#11050 I took note of myself. The XR's side of the class reference is particularly disastrous

@Repiteo Repiteo merged commit 987832b into godotengine:master Jun 24, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Jun 24, 2025

Thanks!

@AThousandShips AThousandShips deleted the fix_doc_spelling branch June 24, 2025 15:14
@AThousandShips
Copy link
Member Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants