hhatto/autopep8

Proposal: Removing --aggressive and --experimental option

hhatto opened this issue · 2 comments

Why?

The current -a, -aa, -aaa, --experimental option is confusing and becomes a factor to increase the maintenance cost. Delete these options and change them to control all operations with --select or --ignore.

migration plan

If you used -a option or --experimental option on the next minor release version of autopep8, it outputs a warning.
And remove the option with the next major update (v2.0).

I would like to hear your opinion.

Thanks.

👍 The best thing about the -a, -aa, etc options was that you can control several classes of operations easily. If the --select and --ignore contain some kind of grouping mechanism I would like the proposed change. As part of this, the documentation would need to be updated to better showcase how the operations are grouped together. Honestly, the documentation needs this anyway so it would be a good reason to dig into it.

As someone new to AutoPEP8 I don't understand the aggressive options. I like the --select, --ignore options, but I really need more guidance. It'd be nice to have a table with a link to each pycodestyle rule and whether autopep8 will fix it by default or I need to add that rule to --select. Maybe sorting the table by default rules vs. non-default would help me focus in on the non-default rules to see which ones I wanted to include.