PrithviBadiga commented on code in PR #66213:
URL: https://github.com/apache/airflow/pull/66213#discussion_r3282321492


##########
airflow-ctl/src/airflowctl/ctl/cli_config.py:
##########
@@ -267,6 +267,27 @@ def _load_help_texts_yaml() -> dict[str, dict[str, str]]:
     type=str,
     help="The Dag ID of the Dag to pause or unpause",
 )
+ARG_DAG_ID_OPTION = Arg(

Review Comment:
   Updated this to follow the generated operations pattern.
   
   I moved the logic into `airflowctl.api.operations` as 
`TasksOperations.state`, removed the hand-written `task_command.py` 
implementation, and let the CLI generate the `tasks state` command from that 
operation.
   
   I also updated the tests accordingly. Since it now follows the standard 
generated-command convention for required primitive parameters, the command 
uses positional arguments:
   `airflowctl tasks state <dag_id> <dag_run_id> <task_id>`



-- 
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]

Reply via email to