-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat: update react & react-dom to v17 #318
Conversation
…rise-public-catalog into bilalqamar95/react-upgrade-to-v17
…rise-public-catalog into bilalqamar95/react-upgrade-to-v17
d49a80d
to
794a783
Compare
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #318 +/- ##
=======================================
Coverage 82.49% 82.49%
=======================================
Files 38 38
Lines 697 697
Branches 219 219
=======================================
Hits 575 575
Misses 118 118
Partials 4 4 ☔ View full report in Codecov by Sentry. |
…rise-public-catalog into bilalqamar95/react-upgrade-to-v17
…rise-public-catalog into bilalqamar95/react-upgrade-to-v17
…se-public-catalog into bilalqamar95/react-upgrade-to-v17
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.
if build is passing then this looks good to me
Double checked, build is passing. Merging this |
Ticket
Upgrade React JS to v17
Description
react
&react-dom
to v17 along withreact-test-renderer
,react-loading-skeleton
,@fortawesome/react-fontawesome
&@testing-library/react
to respective compatible versionsFor all changes
Only if submitting a visual change