tsdx icon indicating copy to clipboard operation
tsdx copied to clipboard

upgrade to parcel-bundler^1.12.5 from parcel^1.12.3 in templates

Open Catminusminus opened this issue 4 years ago • 6 comments
trafficstars

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 (https://github.com/formium/tsdx/issues/980#issuecomment-798875545), so I decided to use not parcel v2 but parcel-bundler.

Catminusminus avatar Jun 03 '21 07:06 Catminusminus

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/formium/tsdx/HzacZy85tBm124gWmtUVknnSFo14
✅ Preview: https://tsdx-git-fork-catminusminus-upgrade-parcelv1-formium.vercel.app

vercel[bot] avatar Jun 03 '21 07:06 vercel[bot]

@Catminusminus something seems to be wrong with the checks. Anything we can do to unlock these?

thebiltheory avatar Aug 05 '21 10:08 thebiltheory

I got the three approvements, so I will explain this situation.

Let me get straight to the point, this project seems to be "dead". See https://github.com/formium/tsdx/issues/1058 for detail. And only you can do Is fork thIs project (or use a forked project like this).

To unlock CI for this PR, an approvement by a reviewer with write access permission is required. And this seems to be impossible now.

Catminusminus avatar Aug 25 '21 07:08 Catminusminus

I got the three approvements, so I will explain this situation.

Let me get straight to the point, this project seems to be "dead". See #1058 for detail. And only you can do Is fork thIs project (or use a forked project like this).

To unlock CI for this PR, an approvement by a reviewer with write access permission is required. And this seems to be impossible now.

thank you

binvb avatar Aug 25 '21 07:08 binvb

Just tried adding parcel bundler with the given version @Catminusminus

Although I can now see this console error,

parcelRequire is not defined

Any ideas what this could be?

sidwebworks avatar Sep 02 '21 17:09 sidwebworks

image

sidwebworks avatar Sep 02 '21 17:09 sidwebworks