ashb commented on a change in pull request #13423:
URL: https://github.com/apache/airflow/pull/13423#discussion_r553280226



##########
File path: airflow/operators/bash_operator.py
##########
@@ -20,7 +20,7 @@
 import warnings
 
 # pylint: disable=unused-import
-from airflow.operators.bash import STDOUT, BashOperator, Popen, gettempdir  # 
noqa
+from airflow.operators.bash import BashOperator  # noqa

Review comment:
       This is probably one of the cases where removing this is _technically_ 
breaking, but people are unlikely to have used it, and it was never documented 
anyway, so is probably fine.
   
   Probably.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to