potiuk commented on issue #6929: [AIRFLOW-6373] Make airflow/utils pylint 
compatible
URL: https://github.com/apache/airflow/pull/6929#issuecomment-569407567
 
 
   @nuclearpinguin ^^ Here you can see  how it helped to add type annotation in 
the BaseOperator-related functions - now BaseOperator can be top-level and can 
be used in method signature.
   
   The next thing will be to move those two BaseOperator methods to 
BaseOperator class. They actually belong there  - helpers are methods that can 
be imported in many places and should be fairly "standalone" so actually I 
think making helper dependent on BaseOperator is bad idea.

----------------------------------------------------------------
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

Reply via email to