Eason09053360 commented on PR #70977: URL: https://github.com/apache/airflow/pull/70977#issuecomment-5301290038
Both applied. Test coverage — added the second logging/log_filename_template rule to the parametrized test, using the suggestion verbatim. I checked that the new case actually pins that specific rule rather than passing on any logging output: dropping just that ConfigChange makes only the new param fail (1 failed, 4 passed), and all five pass once it is restored. Newsfragment — added `airflow-core/newsfragments/70977.bugfix.rst` with the suggested wording, using double backticks for the literal since newsfragments are RST and single backticks render as a title reference rather than inline code. Thanks for the review ! Drafted-by: Claude Code (Opus 5); reviewed by @Eason09053360 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]
