eladkal commented on code in PR #62924:
URL: https://github.com/apache/airflow/pull/62924#discussion_r2891233926
##########
.github/actions/migration_tests/action.yml:
##########
@@ -42,18 +42,12 @@ runs:
airflow db migrate --to-revision heads &&
airflow db downgrade -n 2.7.0 -y &&
airflow db migrate
- # migration tests cannot be run with Python 3.13 now - currently we have
no FAB and no FABDBManager -
- # and airflow (correctly) refuses to migrate things to Airflow 2 when
there is no "ab_user"
- # table created. So migration tests for now will have to be excluded for
Python 3.13 until
- # we start working on 3.2 (with migration to 3.1) or until FAB is
supported in 3.13 (FAB 5)
- # TODO(potiuk) bring migration tests back for Python 3.13 when one of
the two conditions are fulfilled
- if: env.BACKEND != 'sqlite' && inputs.python-version != '3.13'
+ if: env.BACKEND != 'sqlite'
Review Comment:
mmm I think the actual task in the TODO was not handled isn't it?
--
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]
