potiuk commented on a change in pull request #19737:
URL: https://github.com/apache/airflow/pull/19737#discussion_r754407121
##########
File path: scripts/ci/images/ci_run_docker_tests.sh
##########
@@ -0,0 +1,121 @@
+#!/usr/bin/env bash
Review comment:
Yeah but duplicating that will make it more difficult to remove in the
future. I'd prefer to only add python code and remove bash code since we are
going to get rid of bash.
Seems like in this case we are trading one bash for another. Not a good idea.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
