1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
|
Prerequisites to approving a Merge Proposal (MP)
================================================
Over time, it has been found that insufficient testing by reviewers sometimes leads to file manager app trunk not buildable in Qtcreator due to manifest errors, or translation pot file not updated. As such, please follow the checklist below before top-approving a MP.
Checklist
=========
* Does the MP add/remove user visible strings? If Yes, has the pot file been
updated?
* Does the MP change the UI? If Yes, has it been approved by design?
* Did you perform an exploratory manual test run of your code change and any
related functionality?
* If the MP fixes a bug or implements a feature, are there accompanying unit
and autopilot tests?
* Is the file manager app trunk buildable and runnable using Qtcreator?
* Was the debian changelog updated?
* Was the copyright years updated if necessary?
The above checklist is more of a guideline to help file manager app trunk stay buildable,
stable and up to date.
|