feng-tao commented on a change in pull request #4763: [AIRFLOW-3946] Deprecate utils.file.TemporaryDirectory URL: https://github.com/apache/airflow/pull/4763#discussion_r259643327
########## File path: airflow/utils/file.py ########## @@ -23,11 +23,15 @@ import errno import os import shutil +from contextlib import contextmanager Review comment: I see, sorry, miss that before. in that case, could you add line in between `import shutil` and `from contextlib import contextmanager` to distinguish these two categories import. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services