Skip to content

[REFACTOR] correct 'Error Occured' to 'Error Occurred' in scripts/plot.py - #57

Merged
lamarrr merged 1 commit into
lamarrr:mainfrom
Jah-yee:fix/typo-occurred
Jun 29, 2026
Merged

[REFACTOR] correct 'Error Occured' to 'Error Occurred' in scripts/plot.py#57
lamarrr merged 1 commit into
lamarrr:mainfrom
Jah-yee:fix/typo-occurred

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Correct spelling error in user-facing stderr message.

Found via code search: sys.stderr.write("Error Occured")sys.stderr.write("Error Occurred")

@Jah-yee
Jah-yee force-pushed the fix/typo-occurred branch from f94eb16 to 0f61670 Compare June 25, 2026 10:08
@Jah-yee Jah-yee closed this Jun 25, 2026
@Jah-yee Jah-yee reopened this Jun 25, 2026
@Jah-yee

Jah-yee commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

👋 Hi @lamarrr, just following up on this PR. The fix is ready and mergeable — just wanted to check if there are any changes needed or if it can be merged. Thanks!

@danakj

danakj commented Jun 25, 2026

Copy link
Copy Markdown

Why are you harassing someone over a typo like this, with 3 pings in one day?

@Jah-yee

Jah-yee commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Apologies @danakj — you're right, I sent too many pings. I'll stop here and wait for the maintainer to respond at their own pace. Sorry for the spam.

@lamarrr

lamarrr commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Thanks for the pull request.
I'll accept this pull request for now, but if I get multiple subsequent pings in the future, I'll have no choice but to block you @danakj. It is disruptive getting multiple email/github notifications.

@lamarrr lamarrr changed the title fix: correct 'Error Occured' to 'Error Occurred' in scripts/plot.py [REFACTOR] correct 'Error Occured' to 'Error Occurred' in scripts/plot.py Jun 29, 2026
@lamarrr
lamarrr merged commit 8164227 into lamarrr:main Jun 29, 2026
12 of 16 checks passed
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