Skip to content
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

Fixes a bunch of spelling mistakes #5555

Merged
merged 20 commits into from
Jan 28, 2024
Merged

Conversation

Contrabang
Copy link
Contributor

@Contrabang Contrabang commented Jan 26, 2024

About the pull request

Most of the changes here are minor, mostly to comments or descriptions. Here is the full list.

  • recieve -> receive
  • percieve -> perceive
  • alot -> a lot
  • atleast -> at least
  • succesfully -> successfully
  • seperate -> separate
  • noticable -> noticeable
  • concious -> conscious
  • enviroment -> environment
  • existance -> existence
  • foward -> forward
  • interupted -> interrupted
  • independant -> independent
  • guarenteed -> guaranteed
  • overriden -> overridden
  • occuring -> occurring
  • occured -> occurred
  • transciever -> transceiver
  • maintenace -> maintenance

Explain why it's good for the game

Makes it easier to read things

Testing Photographs and Procedure

Screenshots & Videos

Code compiles and runs

Changelog

🆑
spellcheck: Fixed a few uncommon typos across the entire codebase
/:cl:

@Contrabang Contrabang requested a review from fira as a code owner January 26, 2024 01:32
@github-actions github-actions bot added the Grammar and Formatting Fixes the codebase's tpyos and grammatical's errors label Jan 26, 2024
@@ -19,7 +19,7 @@
* * message - The message received on the redis channel.
*/
/datum/redis_callback/proc/on_message(message)
CRASH("on_message not overriden for [type]!")
CRASH("on_message not overridden for [type]!")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change may be notable if specific errors are being tracked across multiple rounds in some kind of database, if so, I can change this back.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is fine.

@LC4492
Copy link
Contributor

LC4492 commented Jan 26, 2024

Also, the maintenance semiotic is also miss-spelled, being called 'maintenace' instead of the correct name. You could look at that too in the meanwhile.

Copy link
Contributor

@Birdtalon Birdtalon left a comment

Choose a reason for hiding this comment

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

Sure.

@Birdtalon Birdtalon added this pull request to the merge queue Jan 28, 2024
Merged via the queue into cmss13-devs:master with commit ee339c1 Jan 28, 2024
26 checks passed
cm13-github added a commit that referenced this pull request Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Grammar and Formatting Fixes the codebase's tpyos and grammatical's errors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants