Closed dulcetti closed 5 years ago
That's a lot of changes. Let me review before I approve the merge.
No problem. The tests don't work yet. I will resolve.
I putted one configuration of eslint. Review and if you can, change it or tell me ;)
@dulcetti This is a ton of code to review. Some of the changes seem unnecessary (csscomb, editorconfig); and some other are nice, but seem unrelated and maybe would be better in a different ticket/PR (lint, testing). I will look into it during the weekend.
@alvaromontoro Well, I thinkthe editorconfig important, but ok.
Yeah, about tests I will create a Issue. You prefer create one PR for each setting? (lint, etc)
19
Add initial config to:
Next features