techieshark
commented
7 years ago $ eslint js/app.js output:
296:10 error Expected indentation of 10 spaces but found 9 indent
297:10 error Expected indentation of 10 spaces but found 9 indent
298:10 error Expected indentation of 10 spaces but found 9 indent
299:10 error Expected indentation of 10 spaces but found 9 indent
300:10 error Expected indentation of 10 spaces but found 9 indent
301:10 error Expected indentation of 10 spaces but found 9 indent
302:10 error Expected indentation of 10 spaces but found 9 indent
303:10 error Expected indentation of 10 spaces but found 9 indent
304:10 error Expected indentation of 10 spaces but found 9 indent
305:10 error Expected indentation of 10 spaces but found 9 indent
306:10 error Expected indentation of 10 spaces but found 9 indent
307:10 error Expected indentation of 10 spaces but found 9 indent
308:10 error Expected indentation of 10 spaces but found 9 indent
309:10 error Expected indentation of 10 spaces but found 9 indent
310:10 error Expected indentation of 10 spaces but found 9 indent
311:10 error Expected indentation of 10 spaces but found 9 indent
312:10 error Expected indentation of 10 spaces but found 9 indent
313:10 error Expected indentation of 10 spaces but found 9 indent
314:10 error Expected indentation of 6 spaces but found 9 indent
316:10 error Expected indentation of 6 spaces but found 9 indent
320:10 error Expected indentation of 6 spaces but found 9 indent
321:10 error Expected indentation of 6 spaces but found 9 indent
✖ 22 problems (22 errors, 0 warnings)
Sweet. A couple things:
1) We'll need some tweaks to make it work with both hover and focus, otherwise it doesn't work for users that are unable to use a mouse. Something like this might do:
2) Running
eslint js/app.jsgives some errors:Looks like a lot, but it's mostly just a few things (suggest using var = list,of,vars; rather than list;of;vars for example).