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

Upgrade babel-plugin-emotion to 9.2.10 #3065

Merged

Conversation

mtzhang
Copy link
Contributor

@mtzhang mtzhang commented Sep 24, 2018

Upgrade babel-plugin-emotion to 9.2.10, which includes an upgrade to touch that resolves a name clash with the system touch. This was causing build errors when trying to touch a directory, which was not fixed until a later major version (diff for touch@1.0.0...touch@2.0.1). Full detail in this issue emotion-js/emotion#861.

Existing tests all still pass, and yarn build, flow, test, lint all run without errors. Please let me know if I need to add additional documentation for this upgrade. I didn't open an issue because it would include much of the same information as this PR, as well as the linked issue in the emotion repo.

See babel-plugin-emotion history for changes between v9.1.2, published April 17, 2018 and v9.2.10, published Sep 19, 2018.

@gwyneplaine
Copy link
Collaborator

🎉thanks for this @mtzhang merged

@gwyneplaine gwyneplaine merged commit 974d625 into JedWatson:master Oct 23, 2018
@mtzhang
Copy link
Contributor Author

mtzhang commented Oct 23, 2018

@gwyneplaine thanks! would be great to get a new release with these changes, whenever convenient 🙏

@mtzhang mtzhang deleted the upgrade/babel-plugin-emotion@9.2.10 branch October 23, 2018 04:20
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.

None yet

2 participants