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

Add support of twisted framework, or possible error is climbing inheritance hierarchy #6

Closed
akshetpandey opened this issue Jan 3, 2014 · 4 comments

Comments

@akshetpandey
Copy link

For this code:

from twisted.internet.protocol import DatagramProtocol

class Class(DatagramProtocol):
    def __init__(self, string, host, port):
        self.transport.write(string, (host, port))

The inspector gives an error that class Class has no method transport. Whereas transport is defined by AbstractDatagramProtocol which is inherited by DatagramProtocol

@jayclassless
Copy link
Contributor

I'm not able to reproduce this issue. What versions of twisted, pylint, and prospector do you have installed?

@akshetpandey
Copy link
Author

I am seeing the issue while using landscape. See this url,

https://landscape.io/github/akshetpandey/pyGameScanner/6/modules/pygamescanner/source.py

@jayclassless
Copy link
Contributor

I think @carlio will need to chime in on this one. I think the problem lies with the environment that Landscape is running prospector in. When twisted is installed, that message doesn't come up. But when run in an environment without twisted, pylint reports that error.

@carlio
Copy link
Member

carlio commented Apr 2, 2014

Thanks for looking into it @jayclassless - you're right, it's about how Landscape runs prospector rather than anything else. Currently, it does not install dependencies of your library, so it will find errors that prospector will not find if you run it locally in your environment which does have the dependencies installed.

I'll close this, as it's not a prospector issue, but rather a landscape issue. I've also opened landscapeio/landscape-issues#31 to keep track of the improvement in Landscape behaviour.

@carlio carlio closed this as completed Apr 2, 2014
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

No branches or pull requests

3 participants