replace call to print with logging.debug

[tinnet]

quick fix for py3 compatibility :)

[jvanasco]

Wow. That should have been a debug for other reasons too. Thanks for catching. Will approve and bump a new release Monday or Tuesday.

On Sep 6, 2015, at 3:07 PM, Martin Contento notifications@github.com wrote:

quick fix for py3 compatibility :)

You can view, comment on, or merge this pull request online at:

#2

Commit Summary

replace call to print with logging.debug
File Changes

M pyramid_debugtoolbar_dogpile/init.py (2)
Patch Links:

https://github.com/jvanasco/pyramid_debugtoolbar_dogpile/pull/2.patch
https://github.com/jvanasco/pyramid_debugtoolbar_dogpile/pull/2.diff
—
Reply to this email directly or view it on GitHub.

[tinnet]

Cool, thanks for getting a new version out quickly, I hate having -e git:/ lines in my requirements files :)

On another note: I tried to make the most minimal change possible here, but if you are interested I can follow it up with a PR that goes over the various pylint warnings in the project?

[jvanasco]

This is merged into the 0.0.5 tag and release on pypi (09/08/2015, 11:04am est)

what sort of warnings are you seeing in pylint that worry you? this passes my flake8 checks -- though I have a handful of settings regarding line length and bracket formatting turned off.

[tinnet]

thanks! :)

most of the warnings are about the single letter variable names I think (i'm at the non-python day job right now, can't check). But if you are not bothered -> let's not touch a running system :)