Closed alejandronanez closed 6 years ago
Thanks but I already use xo for the lint. Why used eslint instead of xo would be better?
Oops, I missed XO on your devDependencies.
Let’s stick with it, XO uses eslint under the hood. I’ll update the PR then.
eslint
but left prettier
.xo
command so we can run XO whenever we needNOTE
After running npm run xo
we got:
But we can address those issues in subsequent PRs
Thanks 😉
Hey @nacimgoura.
I added a couple of things here:
After this, everything should be consistent in terms of code styling.
NOTE: This is using 2 spaces which IMO looks more readable than what we had before, but if you don't like it I can revert this change.
closes #23