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

React <18 does not work due to react-reconciler dependency #1062

Open
1 task
mattrunyon opened this issue Jun 23, 2024 · 5 comments
Open
1 task

React <18 does not work due to react-reconciler dependency #1062

mattrunyon opened this issue Jun 23, 2024 · 5 comments

Comments

@mattrunyon
Copy link

What version of million are you using?

1.0.0-rc.26

Are you using an SSR adapter? If so, which one?

None

What package manager are you using?

npm

What operating system are you using?

Linux

What browser are you using?

Chrome

Describe the Bug

Tried installing @million/lint with a Vite project running React 17.0.2. Can't start anything with errors about trying to import react-dom/client.

react-reconciler has a peer dep for react ^18.3.1. This causes it to be installed by npm in node_modules/@million/lint/node_modules/react. The code to detect the React version looks like it checks via import { version } from 'react' which will resolve to the react version installed by the react-reconciler peer dependency, not the version in the workspace.

Not sure if this is an npm issue only, but the default behavior of npm 7+ is to install peer dependencies that are unmet. So it's expected behavior from npm based on the packages.

What's the expected result?

@million/lint working with React 16.8 - 17 when using npm

Link to Minimal Reproducible Example

https://stackblitz.com/edit/vitejs-vite-xfzbac?file=package.json

Participation

  • I am willing to submit a pull request for this issue.
Copy link

Thanks for opening this issue! A maintainer will review it soon.

@ogi-joo
Copy link

ogi-joo commented Jun 30, 2024

Any update on this?

@mountEvarus
Copy link

@ogi-joo if this helps, reverting to 1.0.0-rc.18 seems to solve the issue for me for the time being

@bingoYB
Copy link

bingoYB commented Jul 1, 2024

@ogi-joo if this helps, reverting to 1.0.0-rc.18 seems to solve the issue for me for the time being

When I switched to 1.0.0-rc.18, a new issue occurred 😢
image

@ogi-joo
Copy link

ogi-joo commented Jul 1, 2024

@mountEvarus @bingoYB
Well I don't use million, but another package which also has problem with react-reconciler.

I use React 19 RC and it doesn't work. Only when I downgrade. I think this is similar problem to mine.

So there has to be some workaround or we wait for react-reconciler patch? Idk

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

4 participants