← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

[Merge] lp:~nskaggs/ubuntu-calculator-app/add-jenkins-info into lp:ubuntu-calculator-app

 

Nicholas Skaggs has proposed merging lp:~nskaggs/ubuntu-calculator-app/add-jenkins-info into lp:ubuntu-calculator-app.

Commit message:
add jenkins info and mp readme

Requested reviews:
  Ubuntu Calculator Developers (ubuntu-calculator-dev)

For more details, see:
https://code.launchpad.net/~nskaggs/ubuntu-calculator-app/add-jenkins-info/+merge/288168

add jenkins info and mp readme
-- 
Your team Ubuntu Calculator Developers is requested to review the proposed merge of lp:~nskaggs/ubuntu-calculator-app/add-jenkins-info into lp:ubuntu-calculator-app.
=== modified file 'README-Mergeproposal.md'
--- README-Mergeproposal.md	2015-12-22 06:32:31 +0000
+++ README-Mergeproposal.md	2016-03-04 20:09:36 +0000
@@ -9,7 +9,7 @@
 *   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, or 
+*   Does the MP change the UI? If Yes, has it been approved by design, or
     discussed with some of the Calculator developers?
 
 *   Did you perform an exploratory manual test run of your code change and any
@@ -26,3 +26,7 @@
 
 The above checklist is more of a guideline to help calculator app trunk stay buildable,
 stable and up to date.
+
+Jenkins
+=======
+In addition to manual reviews, merge proposals are subject to being run in jenkins to ensure the application builds and any unit tests are successful. For more information on jenkins and how it works, see the [Core Apps Jenkins Wiki](https://wiki.ubuntu.com/Touch/CoreApps/Jenkins)
\ No newline at end of file


Follow ups