-
-
Notifications
You must be signed in to change notification settings - Fork 361
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(linting): migration tslint to eslint #820
Conversation
Seems like this will be adjust, looks good to me! |
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.
Spaces ftw
.editorconfig
Outdated
@@ -13,18 +13,18 @@ charset = utf-8 | |||
|
|||
# Indentation override for all JS | |||
[**.js] | |||
indent_style = space | |||
indent_style = tab |
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 would personally use spaces
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.
is this evergreen bikeshed discussion on every project 😅
let's go with tabs :D
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.
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.
Let's agree to use "space: 2", I'm currently fighting to fix "Mixed spaces and tabs" on .vue files.
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.
well, tabs could be space 2 tho :)
then there is another discussion about indentation... 2 or 4 😄
.eslintrc.js
Outdated
'plugin:vue/recommended', | ||
// 'prettier', | ||
], | ||
'env': { |
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.
format 👁️
For me is good to go! |
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.
🚀
Big changes over here.
Usage:
To show problems on all files (.js, .ts, .vue):
To only show errors:
Automatic fix:
Right now we've got 1236 problems (0 errors, 1236 warnings)
Next step would be to add this command on our Github Workflow.
Indentation:
To easily convert tab to space or space to tab you will need to edit .eslintrc.js file & .editorconfig
then you can use
yarn lint --fix
PR type
What's new? (may be part of changelog)