Merge lp://staging/~canonical-platform-qa/qakit/flake8-fixes into lp://staging/~thomir-deactivatedaccount/qakit/trunk

Proposed by Federico Gimenez
Status: Superseded
Proposed branch: lp://staging/~canonical-platform-qa/qakit/flake8-fixes
Merge into: lp://staging/~thomir-deactivatedaccount/qakit/trunk
Diff against target: 428 lines (+65/-68)
11 files modified
qakit/__init__.py (+2/-2)
qakit/commands.py (+8/-7)
qakit/config.py (+2/-2)
qakit/format.py (+18/-12)
qakit/lkkb.py (+11/-9)
qakit/tests/__init__.py (+2/-3)
qakit/tests/test_config.py (+2/-4)
qakit/tests/test_format.py (+5/-6)
qakit/tests/test_trello.py (+3/-6)
qakit/trello.py (+7/-12)
setup.py (+5/-5)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/qakit/flake8-fixes
Reviewer Review Type Date Requested Status
Vincent Ladeuil (community) Approve
Leo Arias (community) Approve
Thomi Richards Pending
Review via email: mp+256674@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2015-04-28.

Commit message

Flake8 fixes

Description of the change

Flake8 fixes

To post a comment you must log in.
Revision history for this message
Leo Arias (elopio) :
review: Approve
Revision history for this message
Vincent Ladeuil (vila) wrote :

@Federico: Thanks for doing this !

Let's get those flake8 issues out of the way and automated asap.

I have a few questions/comments inline but nothing blocking this MP landing.

review: Approve
Revision history for this message
Brendan Donegan (brendan-donegan) :
Revision history for this message
Federico Gimenez (fgimenez) wrote :

Hi Vincent, about the license boilerplate this is text that we are using in other projects like ubuntu-sanity-tests, ubuntu-power-tests and ubuntu-ota-tests, for instance [1]. It comes from the FSF recommendation at [2], but seems that we are using the one-file program recommendation for multi-file programs, we should discuss adopting the right one.

Thanks,

[1] http://bazaar.launchpad.net/~canonical-platform-qa/ubuntu-sanity-tests/trunk/view/head:/ubuntu_sanity_tests/selftests/__init__.py
[2] https://www.gnu.org/licenses/gpl-howto.html

Unmerged revisions

6. By Federico Gimenez

making flake8 happy

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: