-
Notifications
You must be signed in to change notification settings - Fork 508
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
upgrade to parcel-bundler^1.12.5 from parcel^1.12.3 in templates #1040
base: master
Are you sure you want to change the base?
Conversation
…le dirs This will fix 'Invalid Version: undefined' error due to parcel v1
This comment was marked as spam.
This comment was marked as spam.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fixes the issue
@Catminusminus something seems to be wrong with the checks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same error, this commit works for me
I got the three approvements, so I will explain this situation. Let me get straight to the point, this project seems to be "dead". To unlock CI for this PR, an approvement by a reviewer with write access permission is required. |
This comment was marked as resolved.
This comment was marked as resolved.
Just tried adding parcel bundler with the given version @Catminusminus Although I can now see this console error,
Any ideas what this could be? |
This will fix #980.
Currently, in the two template projects (react and react-with-storybook), you will encounter "Invalid Version: undefined" errors when you type
yarn start
in the example directories.This is due to specifying "parcel": "^1.12.3" in package.json and upgrading to 1.12.5 will fix the problem.
However, v1.12.5 exists only in parcel-bundler package, so I change "parcel": "^1.12.3" to "parcel-bundler": "^1.12.5".
In addition, upgrading to parcel v2 beta will break hot module reloading (#980 (comment)), so I decided to use not parcel v2 but parcel-bundler.