Django Pre-flight

Merge lp://staging/~michael.nelson/django-preflight/include-tracebacks into lp://staging/django-preflight

  1. include-tracebacks
  2. Merge into trunk
Proposed by Michael Nelson
Status: Merged
Approved by: Michael Nelson
Approved revision: 32
Merged at revision: 29
Proposed branch: lp://staging/~michael.nelson/django-preflight/include-tracebacks
Merge into: lp://staging/django-preflight
Diff against target: 183 lines (+83/-4)
4 files modified
preflight/management/commands/preflight.py (+3/-0)
preflight/models.py (+8/-1)
preflight/templates/preflight/overview.html (+5/-0)
preflight/tests.py (+67/-3)
To merge this branch: bzr merge lp://staging/~michael.nelson/django-preflight/include-tracebacks
Reviewer Review Type Date Requested Status
Łukasz Czyżykowski (community) Approve
Review via email: mp+203050@code.staging.launchpad.net

Commit message

Include traceback on command line (when verbosity=2) and in comment on overview page.

To post a comment you must log in.
lp://staging/~michael.nelson/django-preflight/include-tracebacks updated
31. By Michael Nelson

Verbosity is a string, no need for new lines in traceback.

32. By Michael Nelson

Use a hidden <pre> instead of comment. Include test for no traceback in view.

Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) :
review: Approve

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