-
Notifications
You must be signed in to change notification settings - Fork 34
Description
Follow up to #140
Note
I'm creating a separate issue since the following changes should not affect package behaviour in user land.
Note
Alike #140 If you are ok with at least one of the proposed changes I can take care of doing separate PR's 😁
Linter
eslint
is installed in devDependencies
but it is not used in npm
scripts or in actions as far as I can tell.
It would be good to use the default configuration combined with https://typescript-eslint.io plugin
Formatter
Alongside with linting it would be good to have also a formatter.
prettier
is widely adopted so I would go for it, alternatively there is biomejs.
Unit test using snapshots
Add more check to GitHub actions
- Add checks for all supported node version
- Add lint and format checks too
Add commit message linter
In order to ease changelog maintenance (#118), commits should follow conventional commits.
Having a commit linter + a git hooks manager will enable commit message validation on user commit:
Examples:
- Linter: https://commitlint.js.org
- git hooks manager: https://typicode.github.io/husky/