Closed allanchau closed 7 years ago
@idearium/developers Finally ready for testing when you get a chance!
@allanchau, could we update it be under the @idearium/eslint-config
namespace rather than esling-config-idearium
?
@smebberson No sure why, but eslint say:
Make sure the module name begins with
eslint-config-
, such aseslint-config-myconfig
.
http://eslint.org/docs/developer-guide/shareable-configs
I think it may be used when you do "extends": "eslint-config-idearium"
or "extends": "idearium"
@allanchau, ah, okay. Do you know if we could use the @idearium
namespace?
@smebberson ah it worked! I just had to update the paths in browser.js
and node.js
.
BTW the linter-eslint in atom seems to be a few versions behind (doesn't support AST selectors) so you will have to install a local copy of eslint when testing. I'll add this to the notes in the description.
@idearium/developers I'm going to merge this in so we can start referencing the git repo rather than individual commits.
This is essentially the initial commit. Please review all files.
Setup
.eslintrc
file with the contents:Testing
Notes
linter-eslint
in atom seems to be a few versions behind (doesn't support AST selectors) so you will have to install a local copy of eslint when testing.