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

Update to v0.14.0-rc3 #21

Merged
merged 5 commits into from
Dec 11, 2020
Merged

Update to v0.14.0-rc3 #21

merged 5 commits into from
Dec 11, 2020

Conversation

JordanMartinez
Copy link

@JordanMartinez
Copy link
Author

I'm just getting this PR started, so it's easier to do node and web libraries.

This commit switches to `Number.fromString` instead of the previous
`Global.readFloat`, which has been deprecated. We use `unsafePartial`
and `fromJust` to recover the prior behavior.

This also removes the `Generic` instances for `HugeNum` and `HugeInt`
as they expose the hidden newtype constructors used for these types.
@thomashoneyman
Copy link

The prior commit switches to Number.fromString instead of the previous Global.readFloat, which has been deprecated. We use unsafePartial and fromJust to recover the prior behavior.

This also removes the Generic instances for HugeNum and HugeInt as they expose the hidden newtype constructors used for these types.

@hdgarrood have an opinion on this?

@thomashoneyman thomashoneyman added the purs-0.14 A reminder to address this issue or merge this PR before we release PureScript v0.14.0 label Dec 9, 2020
@thomashoneyman thomashoneyman merged commit 4c9f38f into purescript-contrib:main Dec 11, 2020
@hdgarrood
Copy link

Sorry I missed this before. This all looks good to me 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
purs-0.14 A reminder to address this issue or merge this PR before we release PureScript v0.14.0
Development

Successfully merging this pull request may close these issues.

3 participants