-
Notifications
You must be signed in to change notification settings - Fork 22
Pull Request Guidelines
Simona Domnisoru edited this page Jun 25, 2024
·
69 revisions
The title of the pull request needs to follow the commitlint pattern. You also need to add the Jira issue number [DHIS2-XXXXX]. An example of such a title would be
-
fix: [DHIS2-8519] field validation bug …
.
By doing so we will have a clear history on the master
branch.
You need at least one approval from the developers.
You need at least one approval from the QA. See more details on the Testing and QA workflow
The reviewer is in charge to click the "Resolve conversation" button once a solution was agreed upon and/or implemented.
The Cypress end-to-end tests run against a real backend instance. If the API has bugs then Cypress tests may fail.
All Cypress tests need to pass for the app to be published.