Skip to content

Untracked: Use StopForUserError instead of stop for user errors#77

Merged
jrwishart merged 3 commits into
masterfrom
untracked_change_stop_to_StopForUserErrors
Oct 3, 2025
Merged

Untracked: Use StopForUserError instead of stop for user errors#77
jrwishart merged 3 commits into
masterfrom
untracked_change_stop_to_StopForUserErrors

Conversation

@jrwishart

Copy link
Copy Markdown
Contributor

No description provided.

@JustinCCYap

Copy link
Copy Markdown
Contributor

Some of these errors are probably meant for us and not the user. Was there a particular one that was causing the bugdups?

@jrwishart

jrwishart commented Oct 3, 2025

Copy link
Copy Markdown
Contributor Author

Some of these errors are probably meant for us and not the user. Was there a particular one that was causing the bugdups?

I'll have to find it in bugdup again but I think I saw the ones with following in the message

  1. empty data
  2. A donut/pie chart could not be created as the input values are all zero.

@JustinCCYap

Copy link
Copy Markdown
Contributor

I'll tag the ones that should still be stop. I think I added this input validation to be extra defensive.

@JustinCCYap

Copy link
Copy Markdown
Contributor

Actually I'll just make the changes myself

@jrwishart jrwishart merged commit 3eb1e4a into master Oct 3, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants