Re: [PR] Add pre-commit to check attrs of TI and TIHistory [airflow]

ephraimbuddy merged PR #40846: URL: https://github.com/apache/airflow/pull/40846 -- 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: commits-unsubscr...@air

Re: [PR] Add pre-commit to check attrs of TI and TIHistory [airflow]

jedcunningham commented on code in PR #40846: URL: https://github.com/apache/airflow/pull/40846#discussion_r1684363093 ## airflow/models/taskinstancehistory.py: ## @@ -44,8 +64,76 @@ class TaskInstanceHistory(Base): dag_id = Column(StringID(), nullable=False) run_id =

Re: [PR] Add pre-commit to check attrs of TI and TIHistory [airflow]

ephraimbuddy commented on code in PR #40846: URL: https://github.com/apache/airflow/pull/40846#discussion_r1682396100 ## airflow/models/taskinstancehistory.py: ## @@ -44,8 +64,76 @@ class TaskInstanceHistory(Base): dag_id = Column(StringID(), nullable=False) run_id = C

[PR] Add pre-commit to check attrs of TI and TIHistory [airflow]

ephraimbuddy opened a new pull request, #40846: URL: https://github.com/apache/airflow/pull/40846 This was requested before, and I saw the value of this method of keeping the attributes in sync. -- This is an automated message from the Apache Git Service. To respond to the message