Merge ppa-dev-tools:make-check-fixes into ppa-dev-tools:main

Proposed by Bryce Harrington
Status: Merged
Merged at revision: c1451af3a7d727b1499275858e4c24f49e82321e
Proposed branch: ppa-dev-tools:make-check-fixes
Merge into: ppa-dev-tools:main
Diff against target: 817 lines (+132/-82)
14 files modified
ppa/debug.py (+3/-0)
ppa/job.py (+9/-6)
ppa/lp.py (+4/-4)
ppa/ppa.py (+11/-7)
ppa/ppa_group.py (+5/-3)
ppa/processes.py (+8/-6)
ppa/result.py (+1/-2)
ppa/subtest.py (+2/-3)
ppa/text.py (+18/-13)
tests/test_lp.py (+11/-3)
tests/test_ppa.py (+3/-3)
tests/test_ppa_group.py (+15/-7)
tests/test_scripts_ppa.py (+40/-22)
tests/test_version.py (+2/-3)
Reviewer Review Type Date Requested Status
Christian Ehrhardt  (community) Approve
Canonical Server Pending
Canonical Server Reporter Pending
Review via email: mp+430005@code.staging.launchpad.net

Description of the change

Cleanup to prep for 0.2 release.

To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Really seems to be just cleanups, whitespace lines, line length, f-string, import order and useless imports ... the stuff you know from your friendly linters.

I didn't see anything odd, therefore +1 to this

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Excellent, thanks

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: