potiuk commented on PR #62656:
URL: https://github.com/apache/airflow/pull/62656#issuecomment-4038932645
@Vamsi-klu
**Issues found:**
- ❌ **Testing Requirements**: This is a bug-fix PR but the diff contains no
new or modified test file — only the source file
`shared/logging/src/airflow_shared/logging/structlog.py` was
changed. The review checklist requires a regression test that fails without
the fix and passes with it. The test plan lists specific tests to run (`pytest
task-sdk/tests/ -k log -v`) but
all checkboxes remain unchecked (- [ ]), indicating the author has not
verified the fix works.
- ⚠ **Gen-AI Disclosure**: The PR was generated by Claude Code (as noted by
the 🤖 footer), but the description does not include the required disclosure
checkbox section ('Was generative
AI tooling used to co-author this PR? - [X] Yes — …'). The informal footer
alone does not satisfy the project's Gen-AI contribution disclosure template.
> **Note:** Your branch is **351 commits behind `main`**. Some check
failures may be caused by changes in the base branch rather than by your PR.
Please rebase your branch and push again
to get up-to-date CI results.
**What to do next:**
- The comment informs you what you need to do.
- Fix each issue, then mark the PR as "Ready for review" in the GitHub UI -
but only after making sure that all the issues are fixed.
- Maintainers will then proceed with a normal review.
--
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]
