Skip to content
This repository was archived by the owner on Apr 17, 2025. It is now read-only.

Conversation

@ruizb
Copy link

@ruizb ruizb commented Feb 28, 2020

  • Remove usage of Object.assign when defining the __assign helper.

@matt2xu
Copy link

matt2xu commented Mar 2, 2020

LGTM, can we merge this now or should we wait for the A/B test of FOR-3813 to be over?

@ruizb
Copy link
Author

ruizb commented Mar 2, 2020

I wonder if it's not better to keep this PR opened forever? 🤔

This way we can keep a visual of all the changes made for our use cases with the files tab. Additionally, the version in our package.json can stay the same, it won't have to change every time we add some changes to this lib. WDYT?

@matt2xu
Copy link

matt2xu commented Mar 2, 2020

Good idea! 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants