Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

upgrade react-scripts to v4 #1878

Merged
merged 4 commits into from
Mar 9, 2021
Merged

upgrade react-scripts to v4 #1878

merged 4 commits into from
Mar 9, 2021

Conversation

willdurand
Copy link
Member

@willdurand willdurand commented Mar 8, 2021

Fixes #1879


resetMocks is false by default in Jest but react-scripts changed it to true, see: jestjs/jest#9047. I'll file an issue to investigate what needs to be done to change resetMocks to true again.

@willdurand willdurand marked this pull request as ready for review March 8, 2021 12:28
@willdurand willdurand requested review from a team and diox and removed request for a team March 8, 2021 12:31
@@ -27,7 +27,6 @@ export const SidePanelBase = ({

return (
<aside
aria-expanded={isExpanded ? 'true' : 'false'}
Copy link
Member Author

Choose a reason for hiding this comment

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

this isn't valid.

@willdurand willdurand merged commit 557b0fd into master Mar 9, 2021
@willdurand willdurand deleted the react-scripts branch March 9, 2021 16:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade react-scripts to v4
2 participants