-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: install and init husky and commitlint #625
Conversation
KUKKUU-UI branch is deployed to platta: https://kukkuu-ui-pr625.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://kukkuu-ui-pr625.dev.hel.ninja 😆🎉🎉🎉 |
f686551
to
c8a31ac
Compare
KK-1347. The Git commits should be linted and the code changes should be tested before the commit is committed. - Installed Husky as a pre commit tool. - Installed commitlint as a tool to check the conventional commit format. - Installed lint-staged as a tool to gather the staged git files. Used this as a guide: https://theodorusclarence.com/shorts/husky-commitlint-prettier.
KK-1347. Install doctoc as development dependency. Run the doctoc during the pre-commit process.
c8a31ac
to
80cb248
Compare
KK-1347. Updated the code styling by running `lint-staged`, that runs `yarn format:code` and `yarn format:scss`.
KUKKUU-UI branch is deployed to platta: https://kukkuu-ui-pr625.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://kukkuu-ui-pr625.dev.hel.ninja 😆🎉🎉🎉 |
481c4b4
to
1a09cfb
Compare
KUKKUU-UI branch is deployed to platta: https://kukkuu-ui-pr625.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://kukkuu-ui-pr625.dev.hel.ninja 😆🎉🎉🎉 |
Doctoc is verbose by default and can take pages just saying that it didn't find anything to update, but it seems there's no normal way to make it less verbose as there's thlorenz/doctoc#261 |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes made by doctoc are left as unstaged changes after the commit has been made, which is not ideal, maybe adding a "Doctoc and how it works"-section or something like that into the README.md and documenting how it works currently would be a transparent way to handle this.
KUKKUU-UI branch is deployed to platta: https://kukkuu-ui-pr625.dev.hel.ninja 🚀🚀🚀 |
TestCafe result is success for https://kukkuu-ui-pr625.dev.hel.ninja 😆🎉🎉🎉 |
I'll add something about the pre-commit hooks in this City-of-Helsinki/kukkuu-admin#302 and this #632. |
KK-1347.
The Git commits should be linted and the code changes should be tested before the commit is committed.
Used this as a guide:
https://theodorusclarence.com/shorts/husky-commitlint-prettier.