-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix formatting and run formatting check in CI #6264
Conversation
Codecov Report
*This pull request uses carry forward flags. Click here to find out more.
|
bb97d27
to
7d4bac4
Compare
7d4bac4
to
219aa59
Compare
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.
Appreciate you not mixing functional changes with the broad formatting ones.
Formatting changes cause code churn and make PRs harder to review. This adds a CI check to ensure formatting is correct on each PR and applies formatting to all the files which were missed in the original move to Prettier.
Future enhancement could auto commit a formatting fix, if needed