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

Use JSBI BigInt instead of Native BigInt #61

Closed
wants to merge 2 commits into from

Conversation

deanpress
Copy link

@deanpress deanpress commented Jan 26, 2020

Summary

Solves #60

This replaces Native BigInt values and methods with JSBI BigInt so front-end web apps can support more browsers, such as Safari on macOS and various other browsers on iOS devices.

Checklist

  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@ghost
Copy link

ghost commented Jan 26, 2020

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.

If no reviewer appears after a week, a reminder will be sent out.

@ghost ghost added the Complexity: Low Less than 64 lines changed. label Jan 26, 2020
@faustbrian
Copy link
Contributor

Closing as explained in #60 (comment)

@faustbrian faustbrian closed this Jan 26, 2020
@ghost
Copy link

ghost commented Jan 26, 2020

Your pull request has been closed, thank you for trying to solve an issue. If you think it was closed prematurely please provide additional information.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Low Less than 64 lines changed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants