singlaamitesh commented on PR #69751: URL: https://github.com/apache/airflow/pull/69751#issuecomment-4946980300
Thanks for the review! 1. Fixed — updated the PR title to drop the conventional-commits-style `fix:` prefix. 2. Done — removed the newsfragment; this is a doc/comment-only link fix with no user-facing behavior change, so it doesn't need one. 3. Understood, and I've updated the PR to use `related: #69164` rather than `closes:` since this PR only addresses the stale en/14 link, not the broader documentation review the issue calls for. Happy to open a follow-up PR for the fuller SQLAlchemy 2.0 doc review if that's still wanted separately. Regarding `airflow dags test` — that line was a leftover template note, not an accurate description of testing for a doc-only YAML change. This is a single-line comment/URL fix in `config.yml` (no code/behavior change), verified by validating the YAML and confirming no other stale `en/14` SQLAlchemy links remain in the repo. I've corrected the PR description to remove the misleading testing note. -- 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]
