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

remove loglevel config from module #98

Merged
merged 1 commit into from Jan 29, 2015

Conversation

Projects
None yet
3 participants
@cpdean
Copy link
Contributor

commented Jan 29, 2015

by always setting the log level, you enforce that log level on every
project that attempts to depend on the pep257 module. instead logging
should be enabled for debug when needed.

this fixes a bug in the flake8-docstrings plugin for flake8

only set loglevel config when running cli
by always setting the log level, you enforce that log level on every
project that attempts to depend on the pep257 module.  instead logging
should be enabled for debug when needed.

this fixes a bug in the flake8-docstrings plugin for flake8

@cpdean cpdean force-pushed the cpdean:master branch from 1e1c86a to 147401d Jan 29, 2015

Nurdok added a commit that referenced this pull request Jan 29, 2015

Merge pull request #98 from cpdean/master
remove loglevel config from module

@Nurdok Nurdok merged commit a5aa2e2 into PyCQA:master Jan 29, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@Nurdok

This comment has been minimized.

Copy link
Member

commented Jan 29, 2015

Thanks @cpdean !

@sigmavirus24

This comment has been minimized.

Copy link
Member

commented Mar 9, 2015

@Nurdok has a version with this included been released?

@Nurdok

This comment has been minimized.

Copy link
Member

commented Mar 10, 2015

Hey @sigmavirus24, sorry for taking so long. This was merged on Jan 29th, but the latest version (0.4.1) was released in Jan 10th, so this PR was not included.

@Nurdok

This comment has been minimized.

Copy link
Member

commented Mar 10, 2015

You can see this information in our new and fancy docs!
http://pep257.readthedocs.org/en/latest/release_notes.html

@sigmavirus24

This comment has been minimized.

Copy link
Member

commented Mar 12, 2015

@Nurdok any chance of an 0.4.2?

@Nurdok

This comment has been minimized.

Copy link
Member

commented Mar 12, 2015

Sure, it might take me a couple of days, though.
On Mar 12, 2015 4:05 AM, "Ian Cordasco" notifications@github.com wrote:

@Nurdok https://github.com/Nurdok any chance of an 0.4.2?


Reply to this email directly or view it on GitHub
#98 (comment).

@Nurdok Nurdok modified the milestone: Next Release Mar 14, 2015

@Nurdok

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.