Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed debug print statement #32

Merged
merged 1 commit into from
Sep 23, 2014
Merged

Removed debug print statement #32

merged 1 commit into from
Sep 23, 2014

Conversation

reidrac
Copy link
Contributor

@reidrac reidrac commented Sep 23, 2014

Also I wonder if you would be open to use Python's "logging" facility instead of the occasional "print", so you could leave debug lines without affecting the code when is not run in debug mode.

Thanks for pywinrm, it is very useful!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.18%) when pulling 04a837f on reidrac:clean-debug-print into d562243 on diyan:master.

diyan added a commit that referenced this pull request Sep 23, 2014
@diyan diyan merged commit e78fc10 into diyan:master Sep 23, 2014
@diyan
Copy link
Owner

diyan commented Sep 23, 2014

@reidrac Sure, "logging" module from Python starndard library must be integrated into pywinrm.

Unfortunately I did not have a time to work on this.

I would be happy to accept any pool requests that add logging support if it follows common/best practices (for example, one of them is "use module-level or class-level loggers and do not use root logger").

@reidrac
Copy link
Contributor Author

reidrac commented Sep 23, 2014

OK, I'll think about it. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants