piston-mini-client

Merge lp://staging/~elachuni/piston-mini-client/debug-envvar into lp://staging/piston-mini-client

  1. debug-envvar
  2. Merge into trunk
Proposed by Anthony Lenton
Status: Merged
Merged at revision: 30
Proposed branch: lp://staging/~elachuni/piston-mini-client/debug-envvar
Merge into: lp://staging/piston-mini-client
Diff against target: 87 lines (+37/-1)
4 files modified
doc/envvars.rst (+6/-0)
doc/index.rst (+2/-1)
piston_mini_client/failhandlers.py (+5/-0)
piston_mini_client/tests/test_failhandlers.py (+24/-0)
To merge this branch: bzr merge lp://staging/~elachuni/piston-mini-client/debug-envvar
Reviewer Review Type Date Requested Status
Ricardo Kirkner Approve
Review via email: mp+54365@code.staging.launchpad.net

Description of the change

Overview
========
This branch makes APIErrors aware of the PISTON_MINI_CLIENT_DEBUG environment variable.

Details
=======
If set, APIErrors will print out the full server response by default, to make debugging easier.
Brief docs and tests were also added.

To post a comment you must log in.
lp://staging/~elachuni/piston-mini-client/debug-envvar updated
31. By Anthony Lenton

Small doc fix and added a test.

Revision history for this message
Ricardo Kirkner (ricardokirkner) wrote :

Very nice work, including the documentation for the change.

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