Merge lp://qastaging/~lool/linaro-image-tools/pep8-fixes into lp://qastaging/linaro-image-tools/11.11

Proposed by Loïc Minier
Status: Merged
Approved by: Deepti B. Kalakeri
Approved revision: 537
Merged at revision: 524
Proposed branch: lp://qastaging/~lool/linaro-image-tools/pep8-fixes
Merge into: lp://qastaging/linaro-image-tools/11.11
Diff against target: 1006 lines (+152/-48)
28 files modified
linaro_image_tools/cmd_runner.py (+2/-2)
linaro_image_tools/hwpack/better_tarfile.py (+1/-0)
linaro_image_tools/hwpack/builder.py (+1/-1)
linaro_image_tools/hwpack/config.py (+3/-5)
linaro_image_tools/hwpack/hardwarepack.py (+1/-1)
linaro_image_tools/hwpack/packages.py (+2/-2)
linaro_image_tools/hwpack/testing.py (+1/-1)
linaro_image_tools/hwpack/tests/__init__.py (+1/-0)
linaro_image_tools/hwpack/tests/test_builder.py (+2/-1)
linaro_image_tools/hwpack/tests/test_hardwarepack.py (+2/-1)
linaro_image_tools/hwpack/tests/test_packages.py (+1/-1)
linaro_image_tools/hwpack/tests/test_script.py (+2/-2)
linaro_image_tools/hwpack/tests/test_testing.py (+1/-0)
linaro_image_tools/media_create/__init__.py (+4/-1)
linaro_image_tools/media_create/boards.py (+8/-6)
linaro_image_tools/media_create/check_device.py (+7/-7)
linaro_image_tools/media_create/chroot_utils.py (+2/-0)
linaro_image_tools/media_create/rootfs.py (+2/-2)
linaro_image_tools/media_create/tests/__init__.py (+1/-0)
linaro_image_tools/media_create/tests/fixtures.py (+2/-0)
linaro_image_tools/media_create/tests/test_media_create.py (+53/-9)
linaro_image_tools/media_create/unpack_binary_tarball.py (+1/-0)
linaro_image_tools/testing.py (+0/-1)
linaro_image_tools/tests/__init__.py (+6/-2)
linaro_image_tools/tests/fixtures.py (+1/-1)
linaro_image_tools/tests/test_cmd_runner.py (+1/-1)
linaro_image_tools/tests/test_pep8.py (+43/-0)
linaro_image_tools/tests/test_pyflakes.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~lool/linaro-image-tools/pep8-fixes
Reviewer Review Type Date Requested Status
linaro-image-tools maintainers Pending
Review via email: mp+109143@code.qastaging.launchpad.net

Description of the change

PEP8 test and compliance fixes.

To post a comment you must log in.
Revision history for this message
Deepti B. Kalakeri (deeptik) wrote :

I have checked the merge changes for the PEP8 compliance and they look good.
But on Natty am not able to run the tests and they fail with http://paste.ubuntu.com/1029987/.
And when I run python -m subunit.run linaro_image_tools.tests.test_pep8 I get the error http://paste.ubuntu.com/1029987/

537. By Loïc Minier

Also silence E202 for natty users with a comment that this can be re-enabled
after natty support is dropped.

Revision history for this message
Deepti B. Kalakeri (deeptik) wrote :

Changes looks good and with revision 537 test_pep8 tests passed.

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