Skip to content

Fix spelling errors in comments and identifiers - #351

Open
NeuronPulse wants to merge 1 commit into
TurboWarp:developfrom
NeuronPulse:develop
Open

Fix spelling errors in comments and identifiers#351
NeuronPulse wants to merge 1 commit into
TurboWarp:developfrom
NeuronPulse:develop

Conversation

@NeuronPulse

Copy link
Copy Markdown

Description:

Fixes a batch of spelling errors found throughout the codebase. Changes are limited to comments, JSDoc, string literals, and a few misspelled internal identifiers — no runtime/serialization behavior is
affected.

Misspellings fixed in comments / JSDoc / string literals

  • recieve → receive (and receieved/recieved → received)
  • occured → occurred (and occurence → occurrence)
  • wether → whether
  • intial → initial
  • compatiblity → compatibility
  • conflcits → conflicts
  • conneciton → connection
  • corropoding → corresponding
  • covnert → convert
  • dependecies → dependencies
  • negitive → negative
  • neumeric → numeric
  • interperated → interpreted
  • amound → amount
  • equivelant → equivalent
  • scatch / scrath → scratch
  • analagous, algoritm, calcuate, fluctations, funtions, genrates, inlcusive, keeing, labling, langauge / languge, lastest, lcoation, mananging, mulitple, multiplcation, objecct, placeholdes, positve,
    pritimive, recursivly, referenes, refrencing, represeting, resolove, resonse, retreive, reuslt, rperesenting, senosr, specificly, superceded, targetting, theads, transcirption, everying, normalzie,
    succesfully, accumalated, activaed

Misspelled internal identifiers renamed

  • CONTORL_INCR_COUNTER → CONTROL_INCR_COUNTER
  • list.instert → list.insert
  • oldCompilerCompatiblity → oldCompilerCompatibility
  • _convertPlaceholdes → _convertPlaceholders

Known issue left intentionally unchanged

  • pen_setPenHueToNumber is not a typo and was left as-is. It is the SB2 legacy IR opcode: sb2_specmap.js maps SB2's setPenHueTo: to it, and it is consumed internally by irgen.js/jsgen.js/enums.js. Renaming
    it would break loading of Scratch 2.0 projects, so no change is made.

Fix misspellings in comments, JSDoc and string literals across the
codebase (e.g. recieve, wether, occured, compatiblity, algoritm).

Rename misspelled internal identifiers:
- CONTORL_INCR_COUNTER -> CONTROL_INCR_COUNTER
- list.instert -> list.insert
- oldCompilerCompatiblity -> oldCompilerCompatibility
- _convertPlaceholdes -> _convertPlaceholders
@CubesterYT

Copy link
Copy Markdown
Member

Was this 100% generated with AI?

@NeuronPulse

Copy link
Copy Markdown
Author

Good question. I did the identifier renames (like CONTORL to CONTROL) manually, those need context and can't be blindly replaced. For comments and docs, I used AI as a spell‑checker since English isn't my first language, but I reviewed every suggestion. So AI only helped with wording, not with code logic.

@GarboMuffin

Copy link
Copy Markdown
Member

maybe it will be merged sometime within the next year but just keep in mind that this kind of pr is slop that is not important to us to get in on any reasonable timeline

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.

3 participants