-
Notifications
You must be signed in to change notification settings - Fork 842
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
Major version dependency upgrades, pt. IV #4112
Conversation
@@ -1,4 +1,6 @@ | |||
{ | |||
"root": true, |
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.
Prevents eslint
from attempting to read this config from EUI root
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
This reverts commit 745d9a0.
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
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.
package.json, yarn.lock, and configuration changes LGTM. Spot checked a few of the src & src-docs changes and they look good.
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
Will wait to merge until after the next release to avoid excessive conflicts in open PRs, possibly closer to the weekend |
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_4112/ |
* eslint; prettier * react-datepicker eslint * prettier autofix * more eslint * CL * Revert "CL" This reverts commit 745d9a0. * lint
Summary
Partial resolution of #3969:
eslint
&prettier
The only change of note is the default configuration update for
prettier
, which now requires that all arrow functions have parentheses around single parameters. We've decided to use the new default value, largely because of type annotations.Checklist
- [ ] Check against all themes for compatibility in both light and dark modes- [ ] Checked in mobile- [ ] Checked in Chrome, Safari, Edge, and Firefox- [ ] Props have proper autodocs- [ ] Added documentation- [ ] Checked Code Sandbox works for the any docs examples- [ ] Added or updated jest tests- [ ] Checked for accessibility including keyboard-only and screenreader modes- [ ] A changelog entry exists and is marked appropriately