Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix serialization issue for 0n. #156

Merged
merged 1 commit into from
Jan 15, 2023
Merged

Fix serialization issue for 0n. #156

merged 1 commit into from
Jan 15, 2023

Conversation

momocow
Copy link
Contributor

@momocow momocow commented Jan 12, 2023

Falsy values except 0n are directly serializable by JSON.stringify; therefore exclude 0n from the early return which checks for falsy values.


Fix #125
Fix #154


I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@okuryu
Copy link
Collaborator

okuryu commented Jan 15, 2023

Thanks!

@okuryu okuryu merged commit 7e23ae8 into yahoo:main Jan 15, 2023
@okuryu
Copy link
Collaborator

okuryu commented Jan 15, 2023

published v6.0.1.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BigInt 0n Throws Error BigInt not supported but it has been since v4
2 participants