You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/13 14:11:30 UTC

[GitHub] [airflow] josh-fell commented on pull request #20226: Fix cached_property MyPy declaration and related MyPy errors

josh-fell commented on pull request #20226:
URL: https://github.com/apache/airflow/pull/20226#issuecomment-992518917


   @potiuk Have you tried the change suggested by @uranusjr [in the thread](https://github.com/apache/airflow/issues/19891#issuecomment-990728228) in #19891 to fix the `cached_property` errors?
   
   ```python
   if sys.version_info >= (3, 8):
       from functools import cached_property
   else:
       from cached_property import cached_property
   ```
   
   @subkanthi tried this and it did not work in his testing (see [comment](https://github.com/apache/airflow/pull/20190#discussion_r767159816)) but I made the change to the Azure provider in my testing and it _did_ resolve the MyPy error.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org