Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR, but I am not seeing any error mentioning "react-dom" when I follow the instructions in the Readme (
npm run build && npm start
).Can you provide a minimal reproduction of the error you are encountering?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jsumners-nr Thank you for wonderful lib and reply.
I got same error when running script dev or build and not sure why you got no errors. Assuming "react-dom" installled globally in your environment?
📌 Reproduction 1:
📌 Reproduction 2:
📌 Error Screenshot:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And I think it's wiered why you got no errors. Next.js requires "react-dom" and there is no "react-dom" in package.json.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. It is a dependency of
next
:Running
npm install
should result in the correct dependencies being installed. Please remove thenode_modules
directory and runnpm install
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for reply. It is weired that when i run this project with
npm
it works. However, when i run it withyarn
it requires react-dom.My yarn version: 1.22.19
How about mention at readme.md that
npm
package manager is suggested to use?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would be okay with a step added between steps 2 and 3 that is "Install dependencies:
npm install
". Would you like to submit a PR for that?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeap. I will close this pr and make new one.