ubuntu-system-settings

Merge lp:~ken-vandine/ubuntu-system-settings/pep8_fix into lp:ubuntu-system-settings

  1. pep8_fix
  2. Merge into trunk
Proposed by Ken VanDine
Status: Merged
Approved by: Jonas G. Drange
Approved revision: 1734
Merged at revision: 1738
Proposed branch: lp:~ken-vandine/ubuntu-system-settings/pep8_fix
Merge into: lp:ubuntu-system-settings
Diff against target: 11 lines (+1/-0)
1 file modified
push-helper/software_updates_helper.py (+1/-0)
To merge this branch: bzr merge lp:~ken-vandine/ubuntu-system-settings/pep8_fix
Reviewer Review Type Date Requested Status
system-apps-ci-bot continuous-integration Needs Fixing
Jonas G. Drange (community) Approve
Review via email: mp+310448@code.launchpad.net

Commit message

Fixed pep8 failure

Description of the change

Fixed pep8 failure

To post a comment you must log in.
Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

LGTM, thanks!

review: Approve
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:1734
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/226/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/1949/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1950
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1789
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1789/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1789/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1789
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/1789/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1789
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1789/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1789/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/1789/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1789/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1789
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1789/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1789
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/1789/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-system-settings-ci/226/rebuild

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'push-helper/software_updates_helper.py'
2--- push-helper/software_updates_helper.py 2014-11-12 12:28:15 +0000
3+++ push-helper/software_updates_helper.py 2016-11-09 15:24:01 +0000
4@@ -183,6 +183,7 @@
5
6 json.dump(obj, open(f2, "w"))
7
8+
9 if __name__ == '__main__':
10 logdir = save_cache_path("ubuntu-system-settings")
11 logfile = os.path.join(logdir, "software_updates_helper.log")

Subscribers

People subscribed via source and target branches