Additional configuration
medic-code opened this issue · comments
@eduardoleao052 Just looking through your code and have a few suggestions to include some additional libraries for config, but might be worth waiting for #6 config to be merged first as there's a large change.
- Prettier and ESlint - for consistent formatting and a linter
- Test infrastructure - Jest is popular and can generate code coverage reports
- lint-staged - linting before commits get added to project
- Husky - running scripts prior to commit/push for tests
Happy to add this in a PR for you to review but let me know how you'd like to proceed.
Hey! Thank you so much for the suggestion! I'm working on including Typescript, which would improve the Repo's reach a lot, but I'd like to keep other libraries to a minimum (the original idea was building it with zero libraries, to be really accessible to any JavaScript learner, but I guess TypeScript is a fair exception).
The good thing with the suggested libraries is that they would just be dev dependencies which are only used to aid development and as part of the build system and tooling so they wouldn't be included in the transpiled output. They're very standard practice for JS and TS projects and they really help make it easier to keep everything consistent and easily understood by other JS/TS developers.
I see! With dev tools, there's really no problem. I just want to keep required libraries for the user to a minimum.
These are good points @pparke, they really do make it easier to keep consistent development and anyone using JS even beginners will almost certainly come across this when creating small or large projects.