-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
add no-unused and react/recommended to eslint #1523
Conversation
fb39d4b
to
4c9c3a3
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.
Thanks, this helps improve code readability a lot
Two points, please consider change:
react.version
in.eslintrc.js
- There is
if (bundled) {}
added insrc/main/kubectl.ts
, I am not sure added by mistake or not.
And these extra two, open for discussion I guess:
- Add https://eslint.org/docs/rules/quotes
- About sub rules of
"unused-imports/no-unused-vars-ts"
, I personally prefer not to use_
, as_
has many different meanings in different programming language. If a variable is not used, I would suggest just remove it, not adding_
, so IMHO these two rules can be removed
"varsIgnorePattern": "^_",
"argsIgnorePattern": "^_",
src/renderer/components/+user-management-roles-bindings/role-binding-details.tsx
Outdated
Show resolved
Hide resolved
src/renderer/item.store.ts
Outdated
@@ -155,7 +155,7 @@ export abstract class ItemStore<T extends ItemObject = ItemObject> { | |||
|
|||
async removeSelectedItems?(): Promise<any>; | |||
|
|||
subscribe(...args: any[]) { | |||
subscribe(..._args: any[]) { |
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.
just remove ...args
is fine.
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.
Even if someone wants to overload this function? I think a follow up PR making this an optional function would be better.
added quotes: double, and remove ignore pattern Signed-off-by: Sebastian Malton <sebastian@malton.name>
4c9c3a3
to
4ab4ccf
Compare
Signed-off-by: Sebastian Malton <sebastian@malton.name>
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.
LGTM
Signed-off-by: Sebastian Malton sebastian@malton.name