keith991001 commented on PR #72857: URL: https://github.com/apache/airflow/pull/72857#issuecomment-5690758613
Thanks for the review! Both points are addressed: - Rebased onto current `main`; the `test_project_structure.py` conflict is resolved. - Consolidated the coverage in b433689: the `default_celery` tests that had accumulated in `test_celery_executor.py` (visibility-timeout warnings, result-backend warning and engine options, broker-transport dict options, `task_acks_late`, and the SSL matrix including the amqps cases) now live in `test_default_celery.py` in the direct `get_default_celery_config` style, deduplicated against the new cases — net −448/+122 lines. Both files pass locally (40 + 32 tests), along with mypy and the project-structure guard. --- Drafted-by: Claude Code (Fable 5); reviewed by @keith991001 before posting -- 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]
