👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines for contributing to lh-avg. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.
This project and everyone participating in it is governed by a Code of Conduct. By participating, you are expected to uphold this code.
- Make sure you always pull the latest changes from the repo and
rebase
from themaster
branch whenever it's necessary. - Always work on the
dev
branch or dedicated branches (e.g.: for features). - Don't modify generated files in built directories.
- Squash commits in branches to reduce the chain whenever its possible.
- Always test your code using
npm test
. - If you add new code, ensure that it's covered by test cases.
- If you fix an issue, mention
fix #x
(where x is the issue number). - If you reference an issue/PR, mention
re #x
. - Ensure that you use the appropriate code style and commit convention.
Make sure the code you're adding has decent test coverage.
Running project tests and coverage:
npm run test
The project uses the commitizen tool for standardizing changelog style commit and a git pre-commit hook to enforce them.