Closed iurisilvio closed 3 days ago
I think it is good to remove extra imports, it is easy to lose control of imports and it cause some overhead.
But if we don't have tooling to find errors caused by conditional imports, it is not a good thing to do. We can do that with flake8 or ruff later.
Description
It fixes a few import issues, found with
ruff check --select="TCH" .
and withimporttime-waterfall
lib.if TYPE_CHECKING
blockType of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?
CI should pass.