Dev-iL commented on PR #74817: URL: https://github.com/apache/airflow/pull/74817#issuecomment-6250072589
Pushed b7c2f189c1 addressing all four inline findings; replies are on the individual threads. On the `Co-Authored-By` trailer in 2ad84a4b — you are right that AGENTS.md rules it out. It sits on an already-pushed commit, so removing it would mean force-pushing the PR branch, and I would rather not rewrite published history for a trailer. Taking the option you offered instead: **it should be dropped from the squash message at merge.** The new commit carries no such trailer. I have also updated the Gen-AI disclosure in the description, since this follow-up was authored with a different model than the original commit. --- Drafted-by: Claude Code (Opus 5) (no human review 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]
