The "Disable Linting" checkbox does nothing
nelstrom opened this issue · 2 comments
nelstrom commented
NullVoxPopuli commented
I think the checkbox only ever did anything if you had ember-cli-eslint or ember-cli-template-lint -- do you have either of those?
If not, we should probs find a way to remove that checkbox!
Thanks for reporting!
nelstrom commented
Ah right! ember-cli-eslint and ember-cli-template-lint are both marked as deprecated.
As I see it, there's two ways of doing this. Either:
- Remove the "Disable Linting" checkbox, and strip out any dead code that handles that functionality
- OR remove the checkbox, but leave the code that implements that functionality, and add some documentation somewhere on how to add the checkbox back for anyone that misses it
Either way, I think the checkbox needs to go. It's squatting on some valuable real-estate.
I've had a go at removing the checkbox in #1140.