Copilot commented on code in PR #67174:
URL: https://github.com/apache/airflow/pull/67174#discussion_r3266038432
##########
airflow-core/src/airflow/executors/workloads/task.py:
##########
@@ -33,8 +33,14 @@
from airflow.models.taskinstancekey import TaskInstanceKey
-class TaskInstanceDTO(BaseModel):
- """Schema for TaskInstance with minimal required fields needed for
Executors and Task SDK."""
+class BaseTaskInstanceDTO(BaseModel):
+ """
+ Base schema for TaskInstance with the minimal fields shared by Executors
and the Task SDK.
+
+ This definition is duplicated in
:mod:`airflow.sdk.execution_time.workloads.task`
Review Comment:
The Sphinx role reference in this docstring has an extra trailing backtick
(``:mod:`airflow.sdk.execution_time.workloads.task````), which will render
incorrectly and may generate doc build warnings. Remove the extra backtick so
the :mod: role is properly closed (and consider adding punctuation after the
reference).
--
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]
