ferruzzi commented on code in PR #37264: URL: https://github.com/apache/airflow/pull/37264#discussion_r1486511527
########## tests/providers/amazon/aws/executors/ecs/test_ecs_executor.py: ########## @@ -103,16 +105,16 @@ def mock_config(): @pytest.fixture def set_env_vars(): Review Comment: Here and many of the changes below: I just moved the repeatedly reused prefix into a constant. Lots of the uses were getting pushed to multiple lines because the prefix was so long and it also makes sense to make that DRY. -- 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]
