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

fix: Fix error thrown after catching null error #7177

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

joeyparrish
Copy link
Member

Sometimes, this catches a "null" error. This shouldn't happen, but if it does, we shouldn't throw yet another exception while handling it.

Sometimes, this catches a "null" error.  This shouldn't happen, but if
it does, we shouldn't throw yet another exception while handling it.
@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

@avelad avelad merged commit b4dc2ad into shaka-project:main Aug 20, 2024
15 of 18 checks passed
@joeyparrish joeyparrish deleted the error-code branch August 20, 2024 17:29
joeyparrish added a commit that referenced this pull request Aug 20, 2024
Sometimes, this catches a "null" error. This shouldn't happen, but if it
does, we shouldn't throw yet another exception while handling it.
joeyparrish added a commit that referenced this pull request Aug 20, 2024
Sometimes, this catches a "null" error. This shouldn't happen, but if it
does, we shouldn't throw yet another exception while handling it.
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Oct 19, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Oct 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants