uranusjr commented on code in PR #42129:
URL: https://github.com/apache/airflow/pull/42129#discussion_r1757312156
##########
airflow/providers/fab/auth_manager/fab_auth_manager.py:
##########
@@ -522,9 +522,13 @@ def _sync_appbuilder_roles(self):
# Otherwise, when the name of a view or menu is changed, the framework
# will add the new Views and Menus names to the backend, but will not
# delete the old ones.
- if conf.getboolean(
- "fab", "UPDATE_FAB_PERMS", fallback=conf.getboolean("webserver",
"UPDATE_FAB_PERMS")
- ):
+ from tests.test_utils.compat import AIRFLOW_V_3_0_PLUS
Review Comment:
This imports a test module in a provider…?? Does not look right to me.
--
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]
