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

Fix #197 by using npm's module to load package.json #199

Closed
wants to merge 1 commit into from

Conversation

nickcmaynard
Copy link
Contributor

Fix #197 by using @npmcli/package-json to load the package.json file for pkg.version comparison purposes.

@Inrixia
Copy link
Owner

Inrixia commented May 1, 2024

Is there no longer support for json imports natively in node? Psure the api has just changed now that the experimental feature is out of beta and a helper package isn't required.

@Inrixia
Copy link
Owner

Inrixia commented May 1, 2024

Closing as this is no longer required with the new build process

@Inrixia Inrixia closed this May 1, 2024
@nickcmaynard
Copy link
Contributor Author

Oh cool - thanks! I thought this was a fairly clean way to deal with the problem, though I see the build changes take care of it now. Looking forward to a push to :latest :-)

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