search

dylang / observatory

Beautiful UI for showing tasks running on the command line.
https://npmjs.org/package/observatory
MIT License
278 stars 17 forks source link

Consider standardjs instead of jshint #10

Open rstacruz opened 8 years ago

rstacruz commented 8 years ago

http://standardjs.com/

I've been using the JS standard style in my packages for a while now. It's a no-semicolon style (hah!), but I've found that it brings quite some amount of sanity. If it's good enough for npm it's good enough for me :)

This may just be a matter of personal style, but I just thought I'd ask for your thoughts.

dylang commented 8 years ago

Sorry, I'm not a fan. Semicolons are part of the language, and the only reason they can be skipped in many circumstances was because when the language was being created, Marc Andreessen suggested it should be as forgiving as HTML to help out new users. To my eyes, skipping ; is like misspelling words on purpose to be cool.

I'm using ESLint via XO in npm-check, I'd be happy to add that.

Here's the settings I use, which I should probably move to another module to make it easier to re-use. https://github.com/dylang/npm-check/blob/master/package.json#L7-L19

rstacruz commented 8 years ago

this isn't part of the discussion but damn, npm-check is pretty cool.

rstacruz commented 8 years ago

also, +1 on XO which is pretty much standard with semicolons.