-
Notifications
You must be signed in to change notification settings - Fork 2
How We Work
JieFu Zhu edited this page Nov 3, 2021
·
17 revisions
- New code has coverage of at least 60%
- All codes pass static inspection and serious problems have been modified. For the rules of static analysis, see
TODO
- All new code is manually reviewed;
- All completed user stories have corresponding test cases;
- Test cases are executed;
- All completed user stories are validated by the Product Owner.
- New functionality documented;
- The function has been accepted by the original requirement submitter;
- open a ticket for every issue/changes you want to make to the codebase
- the main branch should always be a working version - if a merge broke it: revert that merge and open a new merge request
- branch naming rules:
feature_<description>_<date>
,bugfix_<description>_<date>
- branch protection rules:
- before you merge feature branch into main, it has to be reviewed by at least two people
- before you merge code fix branch into main, it has to be reviewed by at least one people
...