search

markmcintyre / brackets-jslint

MIT License
6 stars 5 forks source link

Optionally skip linting whitespace-only lines. #2

Closed Mark-Simulacrum closed 10 years ago

Mark-Simulacrum commented 10 years ago

Follow up on #1, have not tested this.

Does lintWSLines sound like a good option name?

Should the relinting happen on change of this option (I copied the code from the main options to this, since it seems like that makes sense.).

markmcintyre commented 10 years ago

Perhaps let's call it "skipBlankLines" for the sake of user readability, with a default of false. Otherwise this looks good!

Mark-Simulacrum commented 10 years ago

@markmcintyre Should be good now.

Mark-Simulacrum commented 10 years ago

Corrected the if statement.. I agree that the ! is probably countering the option name there.

markmcintyre commented 10 years ago

Works great! Thanks, @Mark-Simulacrum.

TomMalbran commented 10 years ago

Nice Job. Thanks!!