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
chore: dependency updates #14050
chore: dependency updates #14050
Conversation
Codecov Report
@@ Coverage Diff @@
## chore/use-eslint-config #14050 +/- ##
========================================================
Coverage 55.45% 55.45%
========================================================
Files 1275 1275
Lines 31812 31812
Branches 5731 5731
========================================================
Hits 17640 17640
Misses 12355 12355
Partials 1817 1817
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
0806d19
to
791d856
Compare
packages/core/admin/admin/src/components/LanguageProvider/tests/index.test.js
Outdated
Show resolved
Hide resolved
7dec9a1
to
958730a
Compare
bc45a1f
to
81d147c
Compare
@alexandrebodin Looks like we need to add back the |
84f3e37
to
206f12c
Compare
Yes it's supposed to be optional but react-hooks seem to require it |
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.
LGTM!
I'll wait for the eslint PR to be approved to merge this one 👍 |
What does it do?
Why is it needed?
How to test it?
you can try the experimental release of these changes with
npx create-strapi-app@experimental my-app
Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request