Airflint duplicates imports when they already exist in the top-level code

[csp33]

Example:

is refactored to:

(datetime import is duplicated)

[feluelle]

Sorry for the late reply. That is true, we are currently not checking the global usage of the same import.

[Almenon]

Same issue, I would expect it to move the import instead of copying.

[feluelle]

Yes, that is why it is currently recommended to use a tool that can detect unused imports and remove them e.g. autoflake.

Actually, the issue is about not moving to the function level, if it is not possible such as when using the dag constructor as this has to be parsed always.