-
Notifications
You must be signed in to change notification settings - Fork 49
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
craco test
fails
#24
Comments
The fix is here. Looking forward for the author to fix it. |
Thanks for letting me know there's a solution available. |
alistairjcbrown
added a commit
to alistairjcbrown/craco-antd
that referenced
this issue
Jun 8, 2020
This change uses AntD code from `lib/` instead of `es/`, using transpiled code as Jest will not transpile included code from `node_modules`. Styles are also not included. There is no change for built time, which continues to use the module code and pull styles. Fixes DocSpring#10 Fixes DocSpring#24
4 tasks
Has this issue been fixed now that PR #31 has landed and been released? |
I'll try it out shortly. I had been holding off on upgrading for #42. I wasn't sure if the releases on NPM were legitimate or a hacked account. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've updated my package.json as shown in the test app, so I have:
With this change, my test suite no longer runs. I went and checked out the craco-antd test app and I can't get
yarn test
to work there either. It looks like the less compiler isn't being invoked when it should :The text was updated successfully, but these errors were encountered: