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

futurize -a is not affected by fixer setting #114

Open
QuLogic opened this issue Oct 6, 2014 · 0 comments
Open

futurize -a is not affected by fixer setting #114

QuLogic opened this issue Oct 6, 2014 · 0 comments

Comments

@QuLogic
Copy link
Contributor

QuLogic commented Oct 6, 2014

futurize -1 -w -n -a . will add print_function, division, and absolute_import (PS, can those be changed to be in alphabetical order?).

futurize -f libfuturize.fixes.fix_absolute_import -w -n -a . will add the above, plus unicode_literals and future imports.

It would be preferable if, when explicitly specifying a fixer, that only the __future__ imports relevant to that fixer were applied, in much the same way selecting stage 1 (-1) only uses imports that are "safe". This allows one to create smaller commits that would be easier to review and hopefully quicker to be applied.

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