[GitHub] [airflow] potiuk commented on pull request #20979: Remove un-needed deps/version requirements

[GitBox <gi...@apache.org>]

potiuk commented on pull request #20979:
URL: https://github.com/apache/airflow/pull/20979#issuecomment-1017429326


   Very nice. What we really need to remove one more limit. We have another limit in Dockerfile.ci:
   
   ```
   # * lazy-object-proxy<1.5.0: required by astroid
   lazy-object-proxy<1.5.0
   ```
   
   This can likely be removed as wel l(we will find out when we change it. And it might be the root cause of the issue. 
   
   It's been introduced to account for `pip` behaviour with --eager-upgrade (when eager-upgrade is run it will first upgrade "airflow" dependencies without any extras and only then will apply extras which caused similar backtracking because astroid had < 1.5.0 limitation and by the time it was added by extras we already had higher version.
   
   But this limitation is gone and astroid we use now has:
   
   ```
   	lazy_object_proxy>=1.4.0
   ```
   
   So we can safely remove it too.


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