-
Notifications
You must be signed in to change notification settings - Fork 11
Engineering Standards
Song Zheng edited this page Apr 16, 2020
·
5 revisions
A list of engineering standards we follow.
- Make sure to have 100% code coverage. We don't want any uncovered cases due to the nature of the project (databases)
- Only merge when you have 2 or more approvals from admins
- Pick features from the project board. They are ranked by priority.
- Make sure you tag your issue in your commit that. The example below references issue #15, a logging feature
git commit -m 'updates #15: Adds testing to logging module'git commit -m 'closes #15: fixes test cases from code review'