Skip to content
This repository was archived by the owner on Jan 22, 2026. It is now read-only.

Run the styled components babel plugin, replace sucrase with babel#57

Merged
GregWeil merged 8 commits intomasterfrom
babel-styled-components
Oct 23, 2019
Merged

Run the styled components babel plugin, replace sucrase with babel#57
GregWeil merged 8 commits intomasterfrom
babel-styled-components

Conversation

@GregWeil
Copy link
Copy Markdown
Contributor

The styled components plugin doesn't work if anything else has transformed the code before it, so it needs to run in the shared components build step.

Additionally, the plugin's latest version doesn't transpile our code properly, so I've pinned it to the previous version.

I opted to remove sucrase in favor of running everything through babel, but it still works with sucrase handling the jsx transform if that's preferred.

Glitch (shared-components-greg) added 5 commits October 11, 2019 21:43
./package.json:97325/9794
./rollup.config.js:97325/288
./rollup.config.js:97325/228
./rollup.config.js:97325/285
./package.json:97325/3500
./package.json:97325/2
@GregWeil
Copy link
Copy Markdown
Contributor Author

https://quixotic-rocket.glitch.me/ is a working community site with not hydration errors

@modernserf
Copy link
Copy Markdown
Contributor

can you bump the version and update the changelog on this?

@GregWeil
Copy link
Copy Markdown
Contributor Author

Is anything else needed for this pr?

@GregWeil GregWeil merged commit 893b498 into master Oct 23, 2019
@keithk
Copy link
Copy Markdown
Contributor

keithk commented Nov 10, 2020

🚀 PR was released in v0.19.0 🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants