~ubuntu-clock-dev/ubuntu-clock-app/stopwatch-feature-staging

« back to all changes in this revision

Viewing changes to README.mergeproposal

  • Committer: Nekhelesh Ramananthan
  • Date: 2015-08-12 19:16:03 UTC
  • mto: (315.2.13 utopic-3.0)
  • mto: This revision was merged to the branch mainline in revision 358.
  • Revision ID: krnekhelesh@gmail.com-20150812191603-zjjk4j8r91cg73gd
Added mergeproposal checklist readme to help reviewers

Show diffs side-by-side

added added

removed removed

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