Skip to content

fix: correct occured to occurred typo in user-facing error messages (6 instances) - #29

Open
Jah-yee wants to merge 1 commit into
SpeedReflect:masterfrom
Jah-yee:fix/occurred-typo
Open

Jah-yee wants to merge 1 commit into
SpeedReflect:masterfrom
Jah-yee:fix/occurred-typo

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Aug 1, 2026

Copy link
Copy Markdown

Summary

Fixes a typo in user-facing error messages across 4 files (6 total instances).

Files changed:

  • Binary/CLI.cs:92 - Error has occured -> File:...
  • Binary/Program.cs:149 - Unexpected error has occured (MessageBox)
  • Binary/Program.cs:181 - Unexpected error has occured: {e} (MessageBox)
  • Binary/IntroUI.cs:167 - Error has occured -> File:...
  • Binary/Editor.cs:497 - Error has occured -> File:...
  • Binary/Editor.cs:793 - Error has occured -> Line:...

Before: occured → After: occurred

Testing

  • Verified locally: 4 files, 6 insertions(+), 6 deletions(-)
  • All changes are surgical typo corrections only

@Jah-yee

Jah-yee commented Oct 24, 2026

Copy link
Copy Markdown
Author

Following up on this one, as it has been open a while.

I re-checked the current default branch and all six occurrences this PR targets are still present, across four files: Binary/CLI.cs line 92, Binary/Editor.cs lines 497 and 793, Binary/IntroUI.cs line 167, and Binary/Program.cs lines 149 and 181. The branch applies cleanly with no conflicts.

A note for the reviewer: in Program.cs the two occurrences are inside MessageBox.Show calls, so they are visible to end users, while the other four are in the editor's error log strings. I deliberately did not touch the GetLowestMessage() error formatting around them, so the diff stays purely to the spelling.

If this project would prefer a different wording for these messages (they currently read "Error has occured -> File: ... Line: ..."), I can rework the wording, but for now it is a spelling-only change. Let me know what you would like.

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.

1 participant