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
Update common-utils to ESLint 7 #3790
Conversation
common/lib/common-utils/.eslintrc.js
Outdated
], | ||
"parserOptions": { | ||
"project": [ "./tsconfig.json", "./test/tsconfig.json" ] | ||
}, | ||
"rules": { | ||
"@typescript-eslint/no-non-null-assertion": "off", | ||
"@typescript-eslint/no-unsafe-return": "off", | ||
"@typescript-eslint/no-unused-vars": "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.
did these two warnings produce a lot of errors in common-utils/is there an issue to clean them up and remove these warning suppressions?
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.
@heliocliu Thanks - I'm not sure how I wound up suppressing these both at the project level. I fixed that. There aren't too many offenses. I tried to add typings for sha.js and change the import in hashFileNode.ts to import { sha1 } from "sha.js";
but that brings in node types which breaks the build.
If we had proper types for sha.js that would remove a couple of the unsafe returns.
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.
oops thought i approved
This reverts commit 7d17c7c0c4b13d8b34bf46c99ea3307333a41687.
5ed32f4
to
f8005c8
Compare
No description provided.