cloverfield-tools / cf-package

Cloverfield Package Scaffold
MIT License
98 stars 14 forks source link

Use Airbnb Style guide #19

Open nkbt opened 9 years ago

nkbt commented 9 years ago

Our code style is pretty much same as https://github.com/airbnb/javascript so we can switch to https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb ESLint config (and maybe add couple of restrictions/overrides). Should simplify support

ericelliott commented 9 years ago

With a couple important modifications, it's probably good.

I have customized our current .eslintrc quite a bit, I'd have to try AirBnB's version in a lot of projects to see how it works out.

nkbt commented 9 years ago

I meant it to be a base one. Eslint allows to extend/override any config values. Since people usually care about deprecating/updating rules and we won't need to spend time on this kind of maintenance.

ericelliott commented 9 years ago

Cool. :+1:

Let's work up a diff and see what would really have to change to make it happen.