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 lint script and issues #1862
Conversation
"no-undef": "off", | ||
}, | ||
}, | ||
], |
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
Outdated
@@ -61,7 +61,7 @@ | |||
"typedoc": "typedoc --theme markdown --out typedoc --inputFiles ./src/reducers --inputFiles ./src/actions --excludeExternals --excludeNotExported --excludePrivate", | |||
"example-version": "babel-node ./scripts/edit-version.js", | |||
"prettier-all": "prettier --write '{src,examples,test}/**/*.{js,tsx,ts}'", | |||
"lint": "yarn typescript && eslint src/*/src/*.{js,tsx,ts} test webpack examples/*.js examples/**/src/*.{js,tsx,ts} website/*.js website/src --fix", | |||
"lint": "yarn typescript && eslint src/*.js src/**/*.{js,tsx,ts} test webpack examples/*.js examples/**/src/*.{js,tsx,ts} website/*.js website/src --fix", |
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.
Note that for some reason just using src
here caused yarn lint
to run for several hours before I killed it - not sure why, but this seemed preferable.
The build failed lint with 220 additional errors, which I don't understand. Need to investigate why my local env is so different from CI, and then decide whether it's worth fixing all those errors now. |
Signed-off-by: Nick Rabinowitz <nick@unfolded.ai>
Signed-off-by: Nick Rabinowitz <nick@unfolded.ai>
Signed-off-by: Nick Rabinowitz <nick@unfolded.ai>
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.
I was able to yarn
, yarn lint
, and yarn start
without any issues that I could identify. LGTM
As far as I can tell,
yarn lint
has not been checking files insrc
since this change. This fixes theyarn lint
script and addresses the lint failures it reveals.Some of these issues can be resolved by upgrading
eslint
,prettier
, and/oreslint-plugin-prettier
, but for now this just gets lint green.