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

package.json: bump esbuild version to 0.19.1 #405

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

tomasmatus
Copy link
Member

currently building on a fresh main fails with the following error:

❯ make
# if it exists already, npm install won't update it; force that so that we always get up-to-date packages
rm -f package-lock.json
# unset NODE_ENV, skips devDependencies otherwise
env -u NODE_ENV npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR! 
npm ERR! While resolving: cockpit-ostree@undefined
npm ERR! Found: [email protected]
npm ERR! node_modules/esbuild
npm ERR!   dev esbuild@"^0.18.6" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer esbuild@"^0.19.1" from [email protected]
npm ERR! node_modules/esbuild-sass-plugin
npm ERR!   dev esbuild-sass-plugin@"^2.10.0" from the root project
npm ERR! 
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR! 
npm ERR! 
npm ERR! For a full report see:
npm ERR! /home/tmatus/.npm/_logs/2023-08-15T12_42_20_809Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/tmatus/.npm/_logs/2023-08-15T12_42_20_809Z-debug-0.log
make: *** [Makefile:192: package-lock.json] Error 1

@tomasmatus
Copy link
Member Author

image refresh now fails because of this

@mvollmer mvollmer merged commit 6d765c4 into cockpit-project:main Aug 17, 2023
3 checks passed
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