Skip to content
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(lint): lint all files #11727

Closed
wants to merge 2 commits into from
Closed

Conversation

lukekarrys
Copy link
Contributor

This change consolidates all lint rules and paths to the root of the repo, and runs them as part of the lint workflow.

There were only a few manual changes to make, so I opted to make it lint everything, meaning eslint for all *.js files and prettier for all file types it knows about by default. I can easily revert and ignore any files that shouldn't be linted, but as a first pass it was easier to lint everything.

Copy link
Contributor

@iansu iansu left a comment

Choose a reason for hiding this comment

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

Nice! Thanks for doing this.

This change consolidates all lint rules and paths to the root of the
repo, and runs them as part of the `lint` workflow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants