Skip to content

Conversation

@perf2711
Copy link
Contributor

This PR adds two commit hooks:

  • pre-commit - formats all modified files using prettier and lints them using eslint
  • commit-msg - checks if the commit message adheres to prefix: message format

@perf2711 perf2711 added the enhancement New feature or request label Oct 14, 2024
@perf2711 perf2711 requested a review from konraddysput October 14, 2024 11:53
@perf2711 perf2711 self-assigned this Oct 14, 2024
Copy link
Collaborator

@konraddysput konraddysput left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we also need to introduce some guide for this - for example: how to extend the list. We also need to create a CONTRIBUTE guide that explains what we require from commit messages and when

@perf2711
Copy link
Contributor Author

I think we also need to introduce some guide for this - for example: how to extend the list.

I added some docs to commit-msg.js file.

We also need to create a CONTRIBUTE guide that explains what we require from commit messages and when

That's out of scope of this pull request.

@perf2711 perf2711 merged commit f5212a9 into main Nov 13, 2024
@perf2711 perf2711 deleted the repo/commit-hooks branch November 13, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants