Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Switch to different linting tool. #528

Closed
NodeGuy opened this issue Nov 27, 2016 · 2 comments
Closed

Switch to different linting tool. #528

NodeGuy opened this issue Nov 27, 2016 · 2 comments
Assignees
@nivida
Labels
Enhancement Includes improvements or optimizations

Comments

@NodeGuy
Copy link
Contributor

NodeGuy commented Nov 27, 2016

There's extraneous whitespace floating around the code which makes diffs harder to read.

JSHint used to have an flag for warning about extra whitespace but it was removed. Let's switch to a modern linter such as ESLint or even better adopt JavaScript Standard Style instead.
@nivida nivida self-assigned this Aug 9, 2018
@nivida nivida added the Enhancement Includes improvements or optimizations label Aug 9, 2018
@nivida
Copy link
Contributor

nivida commented Nov 29, 2018

I switched to eslint and prettier as you can see here: #2000 :-)

@vibern0
Copy link

vibern0 commented Jan 20, 2019 •
edited
Loading

I switched to eslint and prettier as you can see here: #2000 :-)

Then, should this be moved to done?

@nivida nivida closed this as completed Feb 20, 2019
@alcuadrado alcuadrado mentioned this issue Sep 16, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nivida nivida

Labels
Enhancement Includes improvements or optimizations
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nivida @NodeGuy @vibern0