nuclearpinguin commented on a change in pull request #7456: [AIRFLOW-6817] [WIP] - Move airflow __init__.py imports to sub-packages URL: https://github.com/apache/airflow/pull/7456#discussion_r382734632
########## File path: tests/cli/commands/test_sync_perm_command.py ########## @@ -19,16 +19,16 @@ import unittest from unittest import mock -from airflow import DAG, models from airflow.bin import cli from airflow.cli.commands import sync_perm_command +from airflow.models import DAG, DagBag Review comment: Should we import DAG directly from `airflow.models.dag`? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
