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

Fix using new jsx transform in tests #501

Merged
merged 1 commit into from Dec 4, 2020
Merged

Conversation

msbarry
Copy link
Contributor

@msbarry msbarry commented Oct 27, 2020

The new jsx transform is not working in tests when run through react-app-rewired (they fail with "ReferenceError: React is not defined"). Ported over this change from create-react-app: facebook/create-react-app@2b1161b

The new jsx transform is not working in tests when run through react-app-rewired (the fail with "ReferenceError: React is not defined").  Ported over this change from create-react-app: facebook/create-react-app@2b1161b
@timarney
Copy link
Owner

timarney commented Dec 2, 2020

@dawnmist any thoughts on merging this?

I haven't been keeping from v2 forward.

@dawnmist
Copy link
Collaborator

dawnmist commented Dec 2, 2020

I haven't tried moving from v3 to v4 yet, but the code is matching what react-scripts itself is doing. I have checked it out and tested that things still work properly with react-scripts v3, so it is backwards-compatible/doesn't break things for people who haven't upgraded yet.

@timarney timarney merged commit 2ca3055 into timarney:master Dec 4, 2020
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

3 participants