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

twistedchecker requires camel case even when overriding a method signature. #120

Open
wsanchez opened this issue Dec 7, 2016 · 2 comments

Comments

@wsanchez
Copy link
Member

wsanchez commented Dec 7, 2016

If you are implementing a callable that is meant to have the same signature as a callable in the standard library (or another library which does not use camel case), then twistedchecker will complain.

Because twistedchecker is a required test in Twisted, that means you have to change the signature, which may means it is not compatible in the way intended.

@adiroiban
Copy link
Member

This is a duplicate of #76

@wsanchez
Copy link
Member Author

wsanchez commented Dec 7, 2016

#76 is for method names. This is for arguments, specifically arguments that comply with a non-Twisted interface.

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

No branches or pull requests

2 participants