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 dependency error #883

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

Conversation

naogify
Copy link

@naogify naogify commented Feb 28, 2024

Hi! Thank you for great application. When I run npm install, I got the error below. So, I solve the problem by update @mapbox/mapbox-gl-geocoder@5.0.1 to @5.0.2

Colud you review it When you have time?

$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR! 
npm ERR! While resolving: @mapbox/mapbox-gl-geocoder@5.0.1
npm ERR! Found: mapbox-gl@3.1.2
npm ERR! node_modules/mapbox-gl
npm ERR!   mapbox-gl@"3.1.2" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer mapbox-gl@">= 0.47.0 < 3.0.0" from @mapbox/mapbox-gl-geocoder@5.0.1
npm ERR! node_modules/@mapbox/mapbox-gl-geocoder
npm ERR!   @mapbox/mapbox-gl-geocoder@"^5.0.1" from the root project
npm ERR! 
npm ERR! Conflicting peer dependency: mapbox-gl@2.15.0
npm ERR! node_modules/mapbox-gl
npm ERR!   peer mapbox-gl@">= 0.47.0 < 3.0.0" from @mapbox/mapbox-gl-geocoder@5.0.1
npm ERR!   node_modules/@mapbox/mapbox-gl-geocoder
npm ERR!     @mapbox/mapbox-gl-geocoder@"^5.0.1" 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! /Users/naoppy/.npm/_logs/2024-02-28T19_17_55_411Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /Users/naoppy/.npm/_logs/2024-02-28T19_17_55_411Z-debug-0.log

@naogify
Copy link
Author

naogify commented Feb 28, 2024

But I fond the sentence below from README. If you think it's not necessary, please close this PR.

If you get an error resolving dependencies related to rollup on newer versions of node, then try npm install --force and be sure to not commit changes to package-lock.json.

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.

None yet

1 participant