Add populate_all feature

[adhikasp]

This add --populate-all optional argument to populate (append if exist or create it) __all__ variables with unused modules found in the code.

Closes #16

[coveralls]

Coverage decreased (-0.3%) to 97.908% when pulling 1afce1b on adhikasp:all into 459f5f5 on myint:master.

[jayvdb]

Too generic. --populate-module-dunder-all ?

[adhikasp]

Yeah, but... dunder?

[adhikasp]

Oh I think you mean under

[myint]

I think dunder is good.

[coveralls]

Coverage decreased (-0.05%) to 98.17% when pulling ad4eff0 on adhikasp:all into 459f5f5 on myint:master.

[myint]

Thanks!

I tried this out in a file like:

from foo import bar as alpha, beta

$ ./autoflake.py --pop __init__.py
--- original/__init__.py
+++ fixed/__init__.py
@@ -1 +1,2 @@
 from foo import bar as alpha, beta
+__all__ = ['bar as alpha', 'beta']

And I got an interesting module name. 😄

[adhikasp]

Weird, i test `as` too yesterday and it works :/ Will look into it.

On Thu, 31 Aug 2017, 09:07 Steven Myint ***@***.***> wrote: Thanks! I tried this out in a file like: from foo import bar as alpha, beta $ ./autoflake.py --pop __init__.py --- original/__init__.py +++ fixed/__init__.py @@ -1 +1,2 @@ from foo import bar as alpha, beta +__all__ = ['bar as alpha', 'beta'] And I got an interesting module name. 😄 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#26 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AF3chnXUcDjN1Hr7UeREnsgyrhOBzHWqks5sdhVJgaJpZM4PFTU5> .

-- Regards Adhika Setya Pramudita Undergraduate student of Information Technology University of Gadjah Mada Yogyakarta Web: http://adhikasetyap.me LinkedIn http://id.linkedin.com/in/adhikasp

[myint]

Thanks. You might also want to test imports with tabs in the code:

from alpha import beta<tab>as<tab>gamma

[coveralls]

Coverage increased (+0.09%) to 98.312% when pulling 27a89d9 on adhikasp:all into 459f5f5 on myint:master.

[myint]

Looks good. I found one other edge case. Files like the following, get an empty __all__ added to them.

def foo():
    bar = 0

--- original/foo.py
+++ fixed/foo.py
@@ -1,2 +1,3 @@
 def foo():
     bar = 0
+__all__ = []

[myint]

I guess the other thing I notice from the above is that this new feature affects all files. I think we should be conservative and only touch __init__.py files. This follows #16 (comment).

[coveralls]

Coverage increased (+0.1%) to 98.32% when pulling 310e1a3 on adhikasp:all into 459f5f5 on myint:master.

[coveralls]

Coverage increased (+0.1%) to 98.32% when pulling 7b2b5ae on adhikasp:all into 459f5f5 on myint:master.

[fsouza]

@adhikasp do you still plan to work on this?