nagasrisai commented on PR #64519:
URL: https://github.com/apache/airflow/pull/64519#issuecomment-4195789074
Hi @potiuk, I noticed this PR was closed — I believe it was because the CI
was failing at that point.
All the issues have since been addressed:
- Static checks (ruff, isort): fixed import order and formatting
- Non-DB provider tests: fixed the existing tests to use `return_value=[]`
on mocks so `list(result)` works correctly
- New tests now use `caplog` to assert on warning output rather than the
no-op `pytest.raises`
The latest commit (32fe4166) has a fully green CI run — 65 checks passing,
24 skipped, 0 failures.
Could you reopen the PR if the change still makes sense, or let me know if
it was closed for a different reason? Happy to adjust anything that needs work.
--
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]
