We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Currently the fix_metaclass fixer uses six.with_metaclass:
fix_metaclass
six.with_metaclass
class Foo(Bar): __metaclass__ = Meta
becomes:
import six class Foo(six.with_metaclass(Meta, Bar)): pass
I believe the @six.add_metaclass() decorator is more readable and the fixer might change it to this instead:
@six.add_metaclass()
import six @six.add_metaclass(Meta) class Foo(Bar): pass
However this may have unwanted side effects (read more about them in the pull request introducing the decorator). What do you think?
(I'm willing to work on this and send a PR, but I want to have a consensus here first not to write code that will end up discarded.)
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Currently the
fix_metaclass
fixer usessix.with_metaclass
:becomes:
I believe the
@six.add_metaclass()
decorator is more readable and the fixer might change it to this instead:However this may have unwanted side effects (read more about them in the pull request introducing the decorator). What do you think?
(I'm willing to work on this and send a PR, but I want to have a consensus here first not to write code that will end up discarded.)
The text was updated successfully, but these errors were encountered: