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

Fix import to match pydocstyle v.1.1.0 #191

Merged
merged 3 commits into from
May 2, 2017

Conversation

Nnonexistent
Copy link
Contributor

After recent release of pydocstyle package prospector fails at import PEP257Checker.
This PR corrects import path.

@coveralls
Copy link

coveralls commented Sep 29, 2016

Coverage Status

Coverage decreased (-0.1%) to 59.064% when pulling 6a934db on Nnonexistent:pydocstyle_import_fix into f89d998 on landscapeio:master.

Copy link

@craSH craSH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just ran across this issue as well, and thought I'd comment that this is the same change I'd have suggested in a PR.

$ pip freeze | grep -e prospector -e pydocstyle
prospector==0.12.2
pydocstyle==1.1.0

And from ipython in that environment:

In [1]: from pydocstyle.checker import PEP257Checker, AllError

In [2]: PEP257Checker
Out[2]: pydocstyle.checker.PEP257Checker

In [3]: AllError
Out[3]: pydocstyle.parser.AllError

@jamesdoherty
Copy link

For backwards compatibility, would it be worth importing from pydocstyle.checker first and then falling back to importing from pydocstyle if an ImportError is raised?

@carlio
Copy link
Member

carlio commented Sep 30, 2016

I agree that this approach is the better one to fixing the version, however I merged #190 and made a 0.12.3 release as I didn't have time to thoroughly test this approach yet. I will merge this once I make sure it will work and figure out backwards compatability issues too. Thanks for the really quick PR!

@coveralls
Copy link

coveralls commented Oct 1, 2016

Coverage Status

Coverage increased (+0.07%) to 59.237% when pulling d42dc91 on Nnonexistent:pydocstyle_import_fix into 95e0981 on landscapeio:master.

@carlio carlio merged commit 394859d into landscapeio:master May 2, 2017
@carlio
Copy link
Member

carlio commented May 2, 2017

Finally getting around to cleaning up various PRs and issues, sorry it took a while!

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.

None yet

5 participants