ephraimbuddy commented on PR #49002:
URL: https://github.com/apache/airflow/pull/49002#issuecomment-2855221712
> I've added a field named `test` in `dag` and tried locally with breeze
shell which work well to detect the change.
>
> ```shell
> $ breeze shell "cd airflow-core/src/airflow && alembic check"
>
> [2025-05-06T12:23:10.917+0000] {migration.py:207} INFO - Context impl
SQLiteImpl.
> [2025-05-06T12:23:10.919+0000] {migration.py:210} INFO - Will assume
non-transactional DDL.
> [2025-05-06T12:23:11.124+0000] {compare.py:400} INFO - Detected added
column 'dag.test_field'
> [2025-05-06T12:23:11.158+0000] {messaging.py:71} ERROR - New upgrade
operations detected: [('add_column', None, 'dag', Column('test_field',
String(length=2000), table=<dag>))]
> FAILED: New upgrade operations detected: [('add_column', None, 'dag',
Column('test_field', String(length=2000),
> table=<dag>))]
> Error 255 returned
> ```
>
> but
>
> ```shell
> $ pre-commit run check-model-changes
> Check if model changes require DB
migration..............................Passed
> ```
wow. That's nice
--
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]