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

chore: upgrade Parcel #813

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

chore: upgrade Parcel #813

wants to merge 11 commits into from

Conversation

louisgv
Copy link
Contributor

@louisgv louisgv commented Nov 10, 2023

Details

An initial attempt at upgrading Parcel. Tracking the following blockers:

I need help with:

  • The new parcel rust stuff seems to foobar'ed a lot of path resolution: Invalid path of @parcel\config-default\index.json parcel-bundler/parcel#9319
  • Fixing up the js packager (either rewrite it, patch it, or ping the Parcel devs to fix it)
  • There's a huge issue with the upstream resolver that's been around since 2.9.3, and doesn't seem to be fixed in 2.10.2. Need someone to triage and chase it down with the Parcel folks.
  • Upgrade our HMR pipeline, incorporating some idea from: https://github.com/parcel-bundler/parcel/pull/9068/files (Specifically, with how it handles non-code import. Basically they transformed it into a bundle of the file itself, and a CS to hot reload. Didn't know you can send out 2 bundles at once?!)
  • Test the updated framework out locally

If you're down to help, feel free to ping me on Discord or in the comment section.

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • I agree to license this contribution under the MIT LICENSE
  • I checked the current PR for duplication.

Contacts

  • (OPTIONAL) Discord ID:

If your PR is accepted, we will award you with the Contributor role on Discord server.

To join the server, visit: https://www.plasmo.com/s/d

@louisgv louisgv added help wanted Extra attention is needed need test The issue require further testing, it might have been fixed labels Nov 10, 2023
@louisgv louisgv self-assigned this Nov 10, 2023
@louisgv
Copy link
Contributor Author

louisgv commented Nov 10, 2023

@seawatts
Copy link

seawatts commented Oct 1, 2024

Any progress here? It's blocking #1076

@tbrockman
Copy link
Contributor

tbrockman commented Oct 6, 2024

@seawatts I've started to work on this and have a branch which works for the most part, but it can't really be merged into main as is. Unfortunately there are some issues in the latest versions parcel (all the way up to the latest release, 2.12.0), one which leads to intermittent build failures (fixed in @nightly thankfully) and another which may cause runtime failures due to scope-hoisting removing necessary references (waiting response here parcel-bundler/parcel#9972).

Might be possible to cut a beta release for it though, but that'd be up to @louisgv.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed need test The issue require further testing, it might have been fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants