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

feat(gatsby-plugin-emotion): upgrade plugin for emotion 10 support #10185

Closed
wants to merge 1 commit into from

Conversation

DSchau
Copy link
Contributor

@DSchau DSchau commented Nov 28, 2018

No description provided.

@DSchau
Copy link
Contributor Author

DSchau commented Nov 28, 2018

Ugh, meant to open this vs. my own repo. Closing 😅

@DSchau DSchau closed this Nov 28, 2018
@wKovacs64
Copy link
Contributor

Aw man, finding this in search results had me excited. 🙂 Is that branch in your fork ready to go, @DSchau? If not, anything I can do to help?

@DSchau
Copy link
Contributor Author

DSchau commented Nov 30, 2018

@wKovacs64 feel free to take what I have and run with it! It's mostly done, but it honestly felt a little janky and I wasn't sure if I was doing things in an emotion-recommended way!

@DSchau
Copy link
Contributor Author

DSchau commented Nov 30, 2018

This is the most up-to-date changes -> https://github.com/DSchau/gatsby/pull/3/files

@wKovacs64
Copy link
Contributor

wKovacs64 commented Nov 30, 2018

Anything in particular you felt was unfinished? I upgraded one of my Gatsby sites to v10 using your branch via gatsby-dev and it seemed fine.

Edit: Oh, but I included the jsx pragma manually. Without it, it doesn't seem to work. I'll keep playing.

@wKovacs64
Copy link
Contributor

Moved to #10226

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.

2 participants