potiuk commented on PR #70472:
URL: https://github.com/apache/airflow/pull/70472#issuecomment-5131458945
Heads-up: I merged #69274 earlier today and it has just put this PR into
conflict — apologies, the overlap wasn't visible until I went looking.
#69274 marked the `tasks` CLI commands as migrated, which included adding
exactly the two lines this PR also adds:
- `@deprecated_for_airflowctl("airflowctl tasks list")` on `task_list` in
`airflow-core/src/airflow/cli/commands/task_command.py` (now on `main`)
- `(task_command.task_list, "airflowctl tasks list")` in
`test_command_deprecations.py`
This branch is ~133 commits behind `main`, so both will collide on rebase.
The fix is simply to **drop them from this PR** — they're already on `main` —
and keep the airflowctl side, which is the actual substance here.
Same situation applies to #69397 (`tasks failed-deps`), which I've flagged
there too. Worth knowing the pattern: the AIP-94 marker PRs and the airflowctl
command PRs touch the same two core files, so whichever lands first obliges the
other to rebase.
---
Drafted-by: Claude Code (Opus 5); reviewed by @potiuk 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]
