← Back to branch summary

~ubuntu-branches/ubuntu/trusty/indicator-session/trusty-updates

« back to all changes in this revision

Viewing changes to MERGE-REVIEW

  • Committer: Package Import Robot
  • Author(s): Ubuntu daily release, Robert Ancell, Ted Gould, CI bot
  • Date: 2014-02-07 16:37:27 UTC
  • mfrom: (1.1.75)
  • Revision ID: package-import@ubuntu.com-20140207163727-e3hx2hztitlj9hir
Tags: 12.10.5+14.04.20140207-0ubuntu1
[ Robert Ancell ]
* Move logout into its own section to match design update in
  https://wiki.ubuntu.com/SystemMenu.

[ Ted Gould ]
* Adding acceptance tests and merge review policies.

[ CI bot ]
* Adding acceptance tests and merge review policies

Show diffs side-by-side

added added

removed removed

Lines of Context:
MERGE-REVIEW
 
1
 
 
2
This documents the expections that the project has on what both submitters
 
3
and reviewers should ensure that they've done for a merge into the project.
 
4
 
 
5
== Submitter Responsibilities ==
 
6
 
 
7
 * Ensure the project compiles and the test suite executes without error
 
8
 * Ensure that non-obvious code has comments explaining it
 
9
 * If the change works on specific profiles, please include those in the merge description.
 
10
 
 
11
== Reviewer Responsibilities ==
 
12
 
 
13
 * Did the Jenkins build compile?  Pass?  Run unit tests successfully?
 
14
 * Are there appropriate tests to cover any new functionality?
 
15
 * If the description says this effects the phone profile:
 
16
   * Run tests indicator-session/unity8*
 
17
 * If the description says this effects the desktop profile:
 
18
   * Run tests indicator-session/unity7*
 
19