search

ricardofbarros / linter-js-standard

Atom linter plugin for JavaScript, using JavaScript Standard Style
https://atom.io/packages/linter-js-standard
MIT License
99 stars 48 forks source link

Change error message #189

Closed djake closed 7 years ago

djake commented 7 years ago

Due to #186 I see this message frequently, and so have probably spent way too much time considering its wording...

sonicdoe commented 7 years ago

Did you mean https://github.com/ricardofbarros/linter-js-standard/issues/179 instead of https://github.com/ricardofbarros/linter-js-standard/issues/186? https://github.com/ricardofbarros/linter-js-standard/issues/179 is now fixed in v3.9.2 and you should now (hopefully) never see this warning message.

I’ll consider rewording (or probably even completely removing) this warning message in a future version.

djake commented 7 years ago

I'm just sitting down to do some work after vacation, so this makes me very excited about clicking update. If I have any issues I will report back on the relevant ticket (and will figure out which exactly is the relevant ticket.)

As for this PR, thanks for considering it, it is admittedly nitpicking, but obviously if the underlying bugs are fixed, that is a much bigger win. Thank you!

sonicdoe commented 7 years ago

Nice to hear that, any bug reports are appreciated!

I’ll close this PR for now and, as I said before, consider rewording in a future version.