Optimize unused imports

[nivekcode]

No description provided.

[shaharkazaz]

@kreuzerk I think that this is already covered by the lint rules. are you sure we want to implement it here as well? 🤔

[nivekcode]

I added the issue based on the discussion here: https://twitter.com/ribizlim/status/1277545201255550977

[nivekcode]

But actually you are right. This is already covered by the lint rules. But if its easy to achieve it would be a nice feature.

I imagine a developer who adds an unused import and then commits its changes. The build then fails because of the unused imports. He then would need to fix it and commit again. If import-conductor would offer this feature this would not happen. Since the unused import would be removed during the commit hook.

I see it more as a nice to have feature instead of something that is necessary. What do you think?

[shaharkazaz]

I agree that it's not a necessity for 2 main reasons:

• The main IDEs have built-in support for cleaning unused imports.
• My setup (which I think it's common) includes a pre-commit hook for the lining so I won't be able to commit these cases.

I do agree that it's a nice to have feature and even make our lives easier, I'll add the PRs welcome tag, maybe someone from the community would pitch in 🙂