This is an incomplete list of things to take care while contributing.
- Every commit message should have this format
$project: title
being$project
one ofserver
,web-ui
orinfra
, see the commit list to get a better idea, also, the message should be meaningful and describe what is changed. - Don't touch files or pieces non-related to the commit message, create a different commit instead.
- Keep the commits simple to make the reviews easy.
- Merge commits will be rejected, use rebase instead, run
git config pull.rebase true
after cloning the repository to rebase automatically. - Every commit should have working code with all tests passing.
- Every commit should include tests unless it is not practical.
- We use scalafmt to format the code automatically, follow the IntelliJ setup for scalafmt.
- The pull requests should go to the
develop
branch.
master
branch should never be broken, it contains the current version running in production.
It is simpler to use the recommended developer environment.
- IntelliJ with the Scala plugin.
Use TSLint to keep the code format consistent, please run tslint -c tslint.json 'src/**/*.ts'
and fix the errors before every commit, or use visual code with the Angular Language Service
and TSLint
plugin to see the errors while typing.