potiuk commented on code in PR #46891:
URL: https://github.com/apache/airflow/pull/46891#discussion_r2211828429
##########
airflow-core/tests/unit/api_fastapi/conftest.py:
##########
@@ -126,6 +126,7 @@ def configure_git_connection_for_dag_bundle(session):
login="",
)
session.add(connection)
+ session.commit()
Review Comment:
Hmm. Not realy - we do not have SQLA2 yet - even in Python 3.13 (until we
get FAB 5) - it would be rather difficult to "enable" it like that without
fixing all incompatibilities - which we will only have with
https://github.com/apache/airflow/pull/52233 of yours @Dev-iL
Can't remember - it could be because we had some side-effect from another
test (that could be removed in the meantime). Let me remove it and push latest
changes.
--
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...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
