You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/09/22 03:09:01 UTC

[GitHub] [beam] Abacn commented on pull request #23326: Remove `pip` check in setup.py to allow installation via other package managers (e.g. `poetry`)

Abacn commented on PR #23326:
URL: https://github.com/apache/beam/pull/23326#issuecomment-1254463374

   agree this code is far outdated. 
   On the other side there was recently an issue of pypi dependency backoff with gcp dependencies and got mitigated. Maybe we still want some warning but update the recommended version number and phrasing, and also make sure this does not fail when there is no pypi (add None check or so). @tvalentyn may have more context about this.


-- 
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: github-unsubscribe@beam.apache.org

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