Skip to content

Conversation

@a-c-sreedhar-reddy
Copy link
Contributor

With new version of bs records get compiles to objects.

@MoOx MoOx requested a review from DCKT May 25, 2020 09:06
@DCKT
Copy link
Collaborator

DCKT commented May 25, 2020

This is a breaking change feature, we should release a new npm version

@MoOx Can you add this package on npm 😄 thanks !

@a-c-sreedhar-reddy
Copy link
Contributor Author

@MoOx @phamhoaivu911 made data optional.

Copy link
Member

@MoOx MoOx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@MoOx MoOx merged commit 805367e into rescript-react-native:master Jul 15, 2020
@MoOx
Copy link
Member

MoOx commented Jul 15, 2020

How do you feel we release this as a patch release to keep the version number close to the original lib?
This is definitely a breaking change, but very easy to adjust right?

@phamhoaivu911
Copy link

It's ok for me 👌

@MoOx
Copy link
Member

MoOx commented Aug 11, 2020

2.3.2 is out, sorry for the delay

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants