-
-
Notifications
You must be signed in to change notification settings - Fork 195
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: fix linting #3218
chore: fix linting #3218
Conversation
Commit SHA:4b317966278f79b4e17c121e9af4cc3a63c80ef7 Test coverage results 🧪
|
Commit SHA:4b317966278f79b4e17c121e9af4cc3a63c80ef7 |
FYI the coverage has declined because I removed unused code, so less lines of covered code = lower percentage of code covered. |
eb30eaa
to
080b0de
Compare
|
080b0de
to
e897fc7
Compare
|
Why does this PR exist?
#3142 prepping, in order to add a new lint plugin we need to ensure our linting setup works at least.
What does this pull request do?
Fix some config things, fix some lint errors in the code, adjust config a bit to fit our codebase so I don't have to change too much source code in order to get it passing.
Testing this change
run
yarn lint
locally in this branch