-
Notifications
You must be signed in to change notification settings - Fork 693
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
CRA message should be deprecated #379
Comments
v7 is the latest stable release. Everything looks fine from Serve's end, we should update CRA's docs and CLI. |
Ok, thanks for the feedback |
This is very frustrating on our side. We didn’t add Serve instructions to CRA ourselves. We took a PR from Zeit team. Of course at the time we assumed we can rely on the arguments not changing. Now all CRA apps have non-working hardcoded instructions. |
My sentiment exactly, as that's what happened to me on a production environment and it took me at least 30 minutes to realize and fix the problem |
I filed #382. |
@sulliwane This is now solved, as per this comment. Sorry for the time we've cost you! |
This is what Create React App suggest after running
npm run build
:This is deprecated since V7, as the new api is not compatible at all with previous versions.
I will cross post to CRA github, but I think it's also related to this package somehow. maybe you should consider tagging the V7
@next
instead of@latest
...The text was updated successfully, but these errors were encountered: