rapsealk opened a new pull request, #51500: URL: https://github.com/apache/airflow/pull/51500
Hello, airflow! This pull request makes changes to the `DockerOperator` in the `providers/docker` package, focusing on simplifying environment variable handling and removing unused imports and methods. The most important changes include replacing the `unpack_environment_variables` method with `dotenv_values` for environment variable parsing and removing the unused `StringIO` import. ### Improvements to environment variable handling: * [`providers/docker/src/airflow/providers/docker/operators/docker.py`](diffhunk://#diff-64d67da736d652ac79c5ffdab69e721abb997d337b1dc53f47bb62bb21fba7e0L382-R383): Replaced the `unpack_environment_variables` method with `dotenv_values` for parsing environment variables from `.env` files, simplifying the code and aligning with standard practices. * [`providers/docker/src/airflow/providers/docker/operators/docker.py`](diffhunk://#diff-64d67da736d652ac79c5ffdab69e721abb997d337b1dc53f47bb62bb21fba7e0L517-L527): Removed the `unpack_environment_variables` method entirely, as it is no longer needed. ### Code cleanup: * [`providers/docker/src/airflow/providers/docker/operators/docker.py`](diffhunk://#diff-64d67da736d652ac79c5ffdab69e721abb997d337b1dc53f47bb62bb21fba7e0L28-R28): Removed the unused `StringIO` import from the module. https://github.com/theskumar/python-dotenv/blob/8411987b9301f716245074872afa30646e9b9eb7/src/dotenv/main.py#L367-L400 -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
