Skip to content

Conversation

hmassonn
Copy link

hello, this update of package.json permit using react 17 and pass 100% of npm run test so...

"peerDependencies": {
"d3": "^5.5.0",
"react": "^16.4.1"
"react": "^17.0.2"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok on allowing react 16 or greater, but we shouldn't block clients to use react 16.

Suggested change
"react": "^17.0.2"
"react": ">=16.4.1"

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you could create a new version for this

"eslint-plugin-react": "7.14.3",
"eslint-plugin-standard": "4.0.1",
"html-webpack-plugin": "2.30.1",
"html-webpack-plugin": "^4.0.0",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonder if this does need other dependency updates 🤔

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no

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