← Back to branch summary

~codygarver/+junk/ind-sess

« back to all changes in this revision

Viewing changes to MERGE-REVIEW

  • Committer: Cody Garver
  • Date: 2014-04-03 17:08:08 UTC
  • Revision ID: cody@elementaryos.org-20140403170808-z56s93rorb1dzvmk
Initial import, version 12.10.5+14.04.20140324-0ubuntu1

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