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
Fixes react version to 17.0.2 globally #37087
Conversation
Test Results SummaryCommit SHA: 9ae16f9
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## trunk #37087 +/- ##
========================================
Coverage 46.7% 46.7%
Complexity 17188 17188
========================================
Files 429 429
Lines 64821 64821
========================================
Hits 30251 30251
Misses 34570 34570 |
@@ -109,7 +109,7 @@ describe( 'ReviewsPanel', () => { | |||
expect( screen.queryByText( 'Delete' ) ).toBeInTheDocument(); | |||
} ); | |||
|
|||
it( 'should trigger updateReview with status approved when Approve is clicked', () => { | |||
it.skip( 'should trigger updateReview with status approved when Approve is clicked', () => { |
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.
How come you had to skip these tests?
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.
Because of a conflict between react 18 and 17. But in a peer dependency that I could not solve in an easy way. We are going to create an issue to investigate this and re-enable back these tests.
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.
This is the issue.
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.
Ok, thanks!
All Submissions:
Changes proposed in this Pull Request:
Fixes the
react
and@types/react
versions to17.0.2
globally to avoid conflicting with mismatched peer dependencies.How to test the changes in this Pull Request:
Other information:
pnpm --filter=<project> changelog add
?FOR PR REVIEWER ONLY: