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

Through @atlaskit/tokens and @atlaskit/ds-lib@, @atlaskit/pragmatic-drag-and-drop-react-drop-indicator has a peer dependency on React 18.2.0 #102

Open
tylerlaprade opened this issue Aug 6, 2024 · 9 comments

Comments

@tylerlaprade
Copy link

This is causing warnings every time I install anything with NPM since I'm on React 18.3.0.

npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving: @atlaskit/[email protected]
npm WARN Found: [email protected]
npm WARN node_modules/react
npm WARN   peer react@"^16.3.0 || ^17.0.0 || ^18.0.0" from @ag-grid-community/[email protected]
npm WARN   node_modules/@ag-grid-community/react
npm WARN     @ag-grid-community/react@"32.0.2" from the root project
npm WARN   54 more (...)
npm WARN 
npm WARN Could not resolve dependency:
npm WARN peer react@"^16.8.0 || ^17.0.0 || ~18.2.0" from @atlaskit/[email protected]
npm WARN node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens
npm WARN   @atlaskit/tokens@"^1.56.0" from @atlaskit/[email protected]
npm WARN   node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator
npm WARN 
npm WARN Conflicting peer dependency: [email protected]
npm WARN node_modules/react
npm WARN   peer react@"^16.8.0 || ^17.0.0 || ~18.2.0" from @atlaskit/[email protected]
npm WARN   node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens
npm WARN     @atlaskit/tokens@"^1.56.0" from @atlaskit/[email protected]
npm WARN     node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator
npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving: @atlaskit/[email protected]
npm WARN Found: [email protected]
npm WARN node_modules/react
npm WARN   peer react@"^16.3.0 || ^17.0.0 || ^18.0.0" from @ag-grid-community/[email protected]
npm WARN   node_modules/@ag-grid-community/react
npm WARN     @ag-grid-community/react@"32.0.2" from the root project
npm WARN   54 more (...)
npm WARN 
npm WARN Could not resolve dependency:
npm WARN peer react@"^16.8.0 || ^17.0.0 || ~18.2.0" from @atlaskit/[email protected]
npm WARN node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens/node_modules/@atlaskit/ds-lib
npm WARN   @atlaskit/ds-lib@"^2.3.0" from @atlaskit/[email protected]
npm WARN   node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens
npm WARN 
npm WARN Conflicting peer dependency: [email protected]
npm WARN node_modules/react
npm WARN   peer react@"^16.8.0 || ^17.0.0 || ~18.2.0" from @atlaskit/[email protected]
npm WARN   node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens/node_modules/@atlaskit/ds-lib
npm WARN     @atlaskit/ds-lib@"^2.3.0" from @atlaskit/[email protected]
npm WARN     node_modules/@atlaskit/pragmatic-drag-and-drop-react-drop-indicator/node_modules/@atlaskit/tokens

added 3 packages, removed 1 package, changed 34 packages, and audited 1406 packages in 10s
@alexreardon
Copy link
Collaborator

Just to confirm, this is for the @atlaskit/pragmatic-drag-and-drop-react-drop-indicator package?

@alexreardon alexreardon changed the title Through @atlaskit/tokens and @atlaskit/ds-lib@, this library has a peer dependency on React 18.2.0 Through @atlaskit/tokens and @atlaskit/ds-lib@, @atlaskit/pragmatic-drag-and-drop-react-drop-indicator has a peer dependency on React 18.2.0 Aug 7, 2024
@tylerlaprade
Copy link
Author

tylerlaprade commented Aug 7, 2024

Yes, that's right, version 1.1.2. We're also using pragmatic-drag-and-drop-hitbox and pragmatic-drag-and-drop, but neither of them have this problem.

@alexreardon
Copy link
Collaborator

I'm working on widening the peer dependency ranges react and react-dom in dependencies to be consistent and to include 18.3.0

@alexreardon
Copy link
Collaborator

alexreardon commented Aug 15, 2024

Can you please check again?

All dependent packages should now have the increased react and react-dom peer dependency range of "^16.8.0 || ^17.0.0 || ^18.0.0".

Example: @atlaskit/[email protected]

@tylerlaprade
Copy link
Author

Thanks @alexreardon. Will I need to to manually download and replace the code in my node_modules/? I'm not seeing a new release version yet.

@alexreardon
Copy link
Collaborator

Which package manager are you using? Big goal is to update the transitive dependencies of pdnd to latest

@alexreardon
Copy link
Collaborator

We can cut a new patch version of pdnd if that would be helpful. Could you try removing the dependency and adding it again (hopefully that will ensure that the latest version of transitive dependencies are on latest)

@tylerlaprade
Copy link
Author

I'm using NPM. I keep all version numbers pinned in package.json. When there's a new version, I update the version number and run npm i.

@tylerlaprade
Copy link
Author

After deleting them and re-installing, I no longer see the warnings! Ideally that would have corresponded to a version number change, though, so I (and other users) can just bump the number rather than deleting and reinstalling.

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

No branches or pull requests

2 participants