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

Updates in version 10.9.0 break Parcel compilation #388

Closed
drewbolles opened this issue Apr 6, 2020 · 5 comments
Closed

Updates in version 10.9.0 break Parcel compilation #388

drewbolles opened this issue Apr 6, 2020 · 5 comments

Comments

@drewbolles
Copy link

drewbolles commented Apr 6, 2020

Compiling split.io libraries (both vanilla JS and React) with Parcel throws an error:

TypeError: Incompatible receiver, Map required

I traced this issue to updates made in version in 10.9.0, 10.8.4 compiles fine.

I've created a repo to demonstrate the bug: https://github.com/drewbolles/parcel-split-debug

After cloning run npm i && npm serve and visit the development URL in a browser. You should see the error in the console.

I was unable to figure out the literal cause of the error, but it seems to be related to how Split uses core-js and how Parcel uses it.

@drewbolles drewbolles changed the title Dependency updates in version 10.9.0 break Parcel compilation Updates in version 10.9.0 break Parcel compilation Apr 6, 2020
@chillaq
Copy link

chillaq commented Apr 7, 2020

Thanks for the details, we are checking the issue now.

@smeijer
Copy link

smeijer commented Jun 4, 2020

@chillaq, were you able to find something? I'm having the same issue when using aria-query with parcel. I've created an issue at the parcel repo, but if that's not where the root cause lies, I should post it at the aria-query repo instead.

@chillaq
Copy link

chillaq commented Jun 11, 2020

Hi, which version of Parcel are you using, we are able to repro with parcel 1 version only.

@smeijer
Copy link

smeijer commented Jun 11, 2020

Yeah, it's indeed a parcel 1 issue only. Upgrading to 2 alpha helped.

@EmilianoSanchez
Copy link
Contributor

@drewbolles , sorry for the delay in the notification.

You can upgrade now the JS SDK to version 10.14.0 in your example and it will work fine with Parcel 1, since we have removed core-js dependencies from the SDK (among other optimizations), in order to reduce its size.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants