Bileto

Merge lp://staging/~robru/bileto/gate-on-pylint into lp://staging/bileto

  1. gate-on-pylint
  2. Merge into trunk
Proposed by Robert Bruce Park
Status: Merged
Approved by: Robert Bruce Park
Approved revision: 315
Merged at revision: 313
Proposed branch: lp://staging/~robru/bileto/gate-on-pylint
Merge into: lp://staging/bileto
Diff against target: 235 lines (+39/-34)
9 files modified
Makefile (+3/-1)
run.py (+5/-0)
tests/__init__.py (+1/-0)
tests/test_v1.py (+2/-8)
tickets/app.py (+3/-3)
tickets/disconnect.py (+1/-0)
tickets/models.py (+2/-2)
tickets/sessions.py (+1/-1)
tickets/v1.py (+21/-19)
To merge this branch: bzr merge lp://staging/~robru/bileto/gate-on-pylint
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
CU2D maintainers Pending
Review via email: mp+276801@code.staging.launchpad.net

Commit message

pylint cleanups, start gating MPs on pylint success.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:313
http://jenkins.qa.ubuntu.com/job/bileto-ci/80/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/bileto-ci/80/rebuild

review: Needs Fixing (continuous-integration)
lp://staging/~robru/bileto/gate-on-pylint updated
314. By Robert Bruce Park

Add dep on pylint.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:314
http://jenkins.qa.ubuntu.com/job/bileto-ci/81/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/bileto-ci/81/rebuild

review: Needs Fixing (continuous-integration)
lp://staging/~robru/bileto/gate-on-pylint updated
315. By Robert Bruce Park

Add missing docstring.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:315
http://jenkins.qa.ubuntu.com/job/bileto-ci/82/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/bileto-ci/82/rebuild

review: Approve (continuous-integration)

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