OrganizeImports removes entire lines about rope HOT 1 CLOSED

The signals import was removed because it wasn't used. I don't want it to be removed though, as the import has intentional side effects.

Removing unused import is an intentional behavior. rope can't know that you intend to use an import just for its side effect. Once we've implemented __all__ (#8) you can add the imported lines there to prevent it from being removed. It would also be reasonable if user can add a special comment markers (e.g. # rope: keep-unused-import) on imports lines to prevent rope from removing them during reorganization. I've separated that to a separate ticket (#368).

The User import was removed even though the User module is used in the code.

There is not enough info to reproduce this issue. Please open a new ticket if you find specific scenarios where rope removed imports that it shouldn't have.

from rope.