[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-26 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1242500221 ## tests/models/test_taskmixin.py: ## @@ -0,0 +1,248 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-26 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1242479282 ## tests/models/test_taskmixin.py: ## @@ -0,0 +1,248 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-26 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1241621345 ## tests/models/test_taskmixin.py: ## @@ -0,0 +1,248 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements.

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-23 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1240114070 ## airflow/example_dags/example_setup_teardown_taskflow.py: ## @@ -47,12 +45,10 @@ def normal(): @task_group def section_1(): # You can also

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-22 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1239005511 ## airflow/models/taskmixin.py: ## @@ -69,6 +70,14 @@ def set_downstream( """Set a task or a task list to be directly downstream from the current

[GitHub] [airflow] ephraimbuddy commented on a diff in pull request #32053: Change `as_setup` and `as_teardown` to instance methods

2023-06-22 Thread via GitHub
ephraimbuddy commented on code in PR #32053: URL: https://github.com/apache/airflow/pull/32053#discussion_r1239005511 ## airflow/models/taskmixin.py: ## @@ -69,6 +70,14 @@ def set_downstream( """Set a task or a task list to be directly downstream from the current