kaxil commented on a change in pull request #6950: [AIRFLOW-6392] Remove cyclic dependency baseoperator <-> helpers URL: https://github.com/apache/airflow/pull/6950#discussion_r361996564
########## File path: UPDATING.md ########## @@ -57,6 +57,31 @@ https://developers.google.com/style/inclusive-documentation --> +### Chain and cross_downstream moved from helpers to BaseOperator + +The chain and cross_downstream methods are now moved to airflow.models.baseoperator module from +airflow.utils.helpers module where they belong. + +Helpers module is supposed to contain standalone helper methods that can be imported by all classes. +The `chain` method and `cross_downstream` method both use BaseOperator and if any other package imports +any of the helpers classes it automatically creates an implicit dependency to BaseOperator which Review comment: ```suggestion any of the helpers classes or functions it automatically creates an implicit dependency to BaseOperator which ``` ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
