← Back to branch summary

~kyrofa/junk/pay-service-2

« back to all changes in this revision

Viewing changes to MERGE-REVIEW

  • Committer: CI bot
  • Author(s): Ted Gould
  • Date: 2014-06-18 18:47:04 UTC
  • mfrom: (8.1.10 devel)
  • Revision ID: ps-jenkins@lists.canonical.com-20140618184704-roe2dx6dwvvw37km
Fixes from integration work to make the pay demo come together. 

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
 
 
10
== Reviewer Responsibilities ==
 
11
 
 
12
 * Did the Jenkins build compile?  Pass?  Run unit tests successfully?
 
13
 * Are there appropriate tests to cover any new functionality?
 
14