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

Adds "./package.json" to exports #70

Closed
wants to merge 1 commit into from
Closed

Conversation

subhaze
Copy link

@subhaze subhaze commented Aug 16, 2021

Updates from 1.2.0+ fail when building with Webpack 5/node 16 due to Package subpath './package.json' is not defined by "exports" in [...]/node_modules/vuex-class-modules/package.json

Updates from 1.2.0+ fail when building with Webpack 5/node 16 due to `Package subpath './package.json' is not defined by "exports" in [...]/node_modules/vuex-class-modules/package.json`
@bodograumann
Copy link
Collaborator

First of all, thank you for contributing. It really is appreciated.
At first glance, this solution doesn’t feel right to me though. I have been using vuex-class-modules with webpack-5 (through vue-cli-5) without any problems.
Do you have a minimal reproduction of the problem?

@subhaze
Copy link
Author

subhaze commented Aug 16, 2021

I don't have a minimal setup to share, other than the above mentioning, webpack 5/ Node 16 with a note that it's within a NativeScript setup. It felt wrong to me as well but adding this to the package in node_modules everything builds fine, or, pinning to a version before 1.2.0 builds fine, anything at or after 1.2.0 fails with wanting ./package.json as part of the exports.

When researching this error, most references I found were related to React Native, so, my only thought is that NativeScript/React Native import package.json files within their tooling.

Reference link uuidjs/uuid#449

@subhaze subhaze closed this Nov 5, 2022
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.

2 participants