Skip to content

Wrap Dragon parse logic in try/catch#218

Merged
trueadm merged 3 commits into
mainfrom
catch-parse-error
Apr 15, 2021
Merged

Wrap Dragon parse logic in try/catch#218
trueadm merged 3 commits into
mainfrom
catch-parse-error

Conversation

@trueadm

@trueadm trueadm commented Apr 15, 2021

Copy link
Copy Markdown
Collaborator

Parsing the data might fail, we should account for this.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 15, 2021
@trueadm trueadm merged commit 46cc033 into main Apr 15, 2021
@trueadm trueadm deleted the catch-parse-error branch April 15, 2021 10:32
acywatson pushed a commit that referenced this pull request Apr 9, 2022
* Wrap Dragon parse logic in try/catch

* Format

* Additional typeof check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants