Merge lp:~vthompson/ubuntu-weather-app/weather-updated-icon into lp:ubuntu-weather-app/obsolete.trunk
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Nicholas Skaggs on 2015-10-21 | ||||
| Approved revision: | 414 | ||||
| Merged at revision: | 422 | ||||
| Proposed branch: | lp:~vthompson/ubuntu-weather-app/weather-updated-icon | ||||
| Merge into: | lp:ubuntu-weather-app/obsolete.trunk | ||||
| Diff against target: |
17 lines (+7/-0) 1 file modified
debian/changelog (+7/-0) |
||||
| To merge this branch: | bzr merge lp:~vthompson/ubuntu-weather-app/weather-updated-icon | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jenkins Bot | continuous-integration | Approve on 2015-10-21 | |
| Andrew Hayzen | 2015-05-21 | Approve on 2015-07-13 | |
| Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve on 2015-07-13 | |
|
Review via email:
|
|||
Commit Message
Switch to new icon
Description of the Change
Switch to new icon
| Victor Thompson (vthompson) wrote : | # |
Do we want to land this in the app that is currently in the store?
| Alan Pope 🍺🐧🐱 🦄 (popey) wrote : | # |
We want to land this in the store alongside the other apps. So we need to land it in trunk, certainly. We can then pick our time for store upload.
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
None: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
None: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
None: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
None: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
None: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
| Bartosz Kosiorek (gang65) wrote : | # |
There is an error:
dpkg-source: error: cannot represent change to weather-app@30.png: binary file contents changed
Please cheange an debian/changelog to fix that issue.
| Victor Thompson (vthompson) wrote : | # |
Thanks Bartosz!
PASSED: Continuous integration, rev:414
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Andrew Hayzen (ahayzen) wrote : | # |
Thanks Bartosz for the hint. LGTM, please be nice jenkins :-)
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
None: https:/
PASSED: Continuous integration, rev:414
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
https:/
Executed test runs:
None: https:/
FAILED: Continuous integration, rev:413
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://