Skip to content

fix(package.json): Add React 17 as peer dependency#386

Merged
dlabaj merged 2 commits intopatternfly:mainfrom
rebeccaalpert:peer-dep
Apr 10, 2025
Merged

fix(package.json): Add React 17 as peer dependency#386
dlabaj merged 2 commits intopatternfly:mainfrom
rebeccaalpert:peer-dep

Conversation

@rebeccaalpert
Copy link
Copy Markdown
Member

Added peer dependency - unsure if anything else is needed, but let me know.

@rebeccaalpert rebeccaalpert linked an issue Apr 9, 2025 that may be closed by this pull request
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2025

Deploy Preview for quickstarts ready!

Name Link
🔨 Latest commit 3190c22
🔍 Latest deploy log https://app.netlify.com/sites/quickstarts/deploys/67f6df78d15acf0008b324b1
😎 Deploy Preview https://deploy-preview-386--quickstarts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Copy Markdown
Member

@logonoff logonoff left a comment

Choose a reason for hiding this comment

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

"@patternfly/react-core": "^6.0.0",
"react": ">=18.0.0",
"react": "17.0.0 || >=18.0.0",
"react-dom": ">=18.0.0",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"react-dom": ">=18.0.0",
"react-dom": "^17 || ^18",

Co-authored-by: logonoff <git@logonoff.co>
@dlabaj dlabaj merged commit 64135e9 into patternfly:main Apr 10, 2025
9 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 6.3.0-prerelease.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug - Quickstarts should have React 17 listed as a peer dependency

3 participants