question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Should lintOnSave be renamed?

See original GitHub issue

I actually do want lint on save, which I have configured through my editor, but I choose to keep the lintOnSave option in vue.config.js disabled because it does more than that - it halts compilation on any linting errors (even if it’s a minor stylistic issue). I’ve seen this behavior frustrate some as they didn’t expect it, so perhaps we should rename the option to better match what it actually does. haltCompilationOnLintingErrors might be a little unwieldly, but I’m open to other ideas.

Another option might be to always run the linter, but only if compilation fails due to a syntax error. I don’t know if eslint-loader actually allows that or if it can be done some other way, but I think that’s more helpful behavior that people are probably less likely to want to disable.

Issue Analytics

  • State:closed
  • Created 6 years ago
  • Comments:6 (6 by maintainers)

github_iconTop GitHub Comments

9reactions
yyx990803commented, Feb 9, 2018

What about allowing lintOnSave to take values such as 'error' and 'warning'? With lint errors being emitted as warnings it won’t halt compilation.

0reactions
LinusBorgcommented, Feb 9, 2018

Agree

Read more comments on GitHub >

github_iconTop Results From Across the Web

Configuration Reference | Vue CLI
To make lint errors show up in the browser overlay, you can use lintOnSave: 'default' . This will force eslint-loader to actually emit ......
Read more >
Component name "Temp" should always be multi-word vue ...
Your linter just tells you, that your component name should be a multi word like MyTemp instead of just Temp. You could rename...
Read more >
ESLint - Quasar Framework
Having a code linter (like ESLint in place is highly recommended and ensures your code looks legible. It also helps you capture some...
Read more >
@vue/cli-plugin-eslint | Yarn - Package Manager
To make lint errors show up in the browser overlay, you can use lintOnSave: 'error' . ... vue invoke should delete renamed/removed files...
Read more >
Go with Visual Studio Code
You can install the Go extension from the VS Code Marketplace. ... lintOnSave setting and configuring your choice of linting tool ( staticcheck ......
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found