Skip to content
This repository has been archived by the owner on Aug 20, 2018. It is now read-only.

Extend a base config #5

Open
jstcki opened this issue Feb 2, 2016 · 3 comments
Open

Extend a base config #5

jstcki opened this issue Feb 2, 2016 · 3 comments
Labels

Comments

@jstcki
Copy link
Contributor

jstcki commented Feb 2, 2016

It's kinda hard to keep up with all eslint features, so maybe we should just extend eslint:recommended? Then it also becomes more clear which are our overrides.

Cf. https://github.com/rackt/eslint-config-rackt

@jstcki jstcki added the question label Feb 2, 2016
@jstcki
Copy link
Contributor Author

jstcki commented Mar 10, 2016

How about depending on and extending https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb, since they're actively maintaining it. We've already used it as a base when we started this config. We made some changes for our own needs but 95% is airbnb style. Another advantage would be that they also maintain a style guide and that their config is in line with that.

@jstcki jstcki added this to the 3.0 milestone Mar 10, 2016
@jstcki
Copy link
Contributor Author

jstcki commented Mar 10, 2016

FWIW, that would also solve #6.

@jstcki jstcki removed this from the 3.0 milestone May 31, 2016
@grossbart
Copy link

I might look into this, because we would actually need to be able to solve #6 in our current project where Purescript interop is only available through ES5, not ES6. Because the foreign function interface is also the place where the compiler can help the least and where most bugs are to be expected, a solid linter would be necessary. We also have other projects that are purely ES5 and that we will maintain for a while longer.

Is there anything I need to know with regards to trying to solve this issue? Still not sure whether tackling this is worth it, but if the problem keeps bugging me, it might be 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants