Skip to content
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

Improve dev-x with prettier and husky hooks #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mochitto
Copy link
Contributor

Resolve #75
To have consistent formatting across the project and facilitate maintenance and contributions I've added prettier (with an import-statements sorting plugin to improve readability) and git-hooks support with husky.

Git hooks with husky formats code and blocks commits until all the desired conditions are met.
The current configuration formats, lints and tests code, before allowing users to commit changes.

Further improvements to the dev-x and contributions could be added by making the repo commitizen friendly


The above message has been included in the commit's description.
The changelog has been updated to reflect the changes, with the version Unreleased to facilitate library moderation.
The prettier configuration mimicks the state of the current codebase.

The added dependencies are development dependencies.

After merging, make sure to run yarn format project-wise. CHANGELOG has been exempted from formatting, although eligible.

Resolve Strumenta#75
To have consistent formatting across the project and facilitate
maintenance and contributions I've added `prettier` (with an
*import-statements* sorting plugin to improve readability) and
*git-hooks* support with `husky`.

*Git hooks* with `husky` formats code and blocks commits until all the
desired conditions are met.
The current configuration formats, lints and tests code, before allowing
users to commit changes.

Further improvements to the dev-x and contributions could be added by
making the repo [commitizen friendly](https://github.com/commitizen/cz-cli)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dev-X and contributions: Prettier and eslint configurations
1 participant