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

Remove reference to nonfunctional third-party package #1067

Open
wants to merge 1 commit into
base: v2
Choose a base branch
from

Conversation

benwilliamsBJSS
Copy link

I get @parcel/package-manager: Could not find module "parcel-resolver-elm-bundle", but it was listed in package.json. Run your package manager first. whenever attempting to utilise the package recommended.

Removing so future developers don't lose as much hair as I did.

I get  `@parcel/package-manager: Could not find module "parcel-resolver-elm-bundle", but it was listed in package.json. Run your package
manager first.` whenever attempting to utilise the package recommended.

Removing so future developers don't lose as much hair as I did.
@vercel
Copy link

vercel bot commented Dec 2, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
website ✅ Ready (Inspect) Visit Preview Dec 2, 2022 at 1:58AM (UTC)

@mischnic
Copy link
Member

mischnic commented Dec 2, 2022

@ChristophP Could you please:

  • Create a github repo for that npm package and
  • also fix the package.json main field which currently points to a non-existent index.js where it should be src/resolver.js

@ChristophP
Copy link
Contributor

@benwilliamsBJSS sorry if this costs you time. This got buried under some other todos.

@mischnic I published a repo here https://github.com/ChristophP/parcel-resolver-elm-bundle . Also I cleaned up some things in the package, fixed the entrypoint and published a new version. This should be set now.

@ChristophP
Copy link
Contributor

Still found one bug in it and also ran into this parcel-bundler/parcel#7107 .
Now I published version 1.0.2 which worked for me and should work for hopefully everyone.

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

Successfully merging this pull request may close these issues.

3 participants