Flag to skip for __init__.py files

[ErikBjare]

Since __init__.py files tend to have "unused" imports, autoimport will happily remove them.

This is fine if the script is run from somewhere where we can easily filter out the __init__.py (like in a Makefile), but becomes unwieldy when run through "fix-on-save" in your editor.

autoflake has solved the same problem with a --ignore-init-module-imports flag, perhaps we could add the same to autoimport?

Edit: should've opened this as a feature request, sry.

[lyz-code]

Hi @ErikBjare thanks for taking the time to open not only an issue but also for making the PR. I'm fine with merging it but just a question. If you do imports in __init__.py files, shouldn't you also add the imported objects to the __all__ variable? If you do so, then autoimport won't delete the imports.

I guess there are more keystrokes for the same result, although for me it adds clarity.

BTW, it's an honor to have your name under the contributors of the project, I <3 activitywatch!

[lyz-code]

Available since 1.3.0 thanks to @ErikBjare

[ErikBjare]

Thanks for the swift merge!

If you do imports in init.py files, shouldn't you also add the imported objects to the all variable? If you do so, then autoimport won't delete the imports.

I guess there are more keystrokes for the same result, although for me it adds clarity.

Yeah, didn't think of that as a workaround, but I almost never use __all__ and don't really have a good reason to change it in all my codebases.

BTW, it's an honor to have your name under the contributors of the project, I <3 activitywatch!

Yay, that makes me happy. Thank you ❤️