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/11/22 15:49:09 UTC

[GitHub] [beam] BjornPrime opened a new issue, #24312: Update Python version in .pre-commit-config.yaml

BjornPrime opened a new issue, #24312:
URL: https://github.com/apache/beam/issues/24312

   ### What needs to happen?
   
   Current `default_language_version` for Python is set to 3.7, which is causing pre-commit check to throw errors when they're run in non-3.7 environments. Updating this will streamline development in more recent versions of Python.
   
   ```
   [INFO] Installing environment for https://github.com/pre-commit/mirrors-yapf.
   [INFO] Once installed this environment will be reused.
   [INFO] This may take a few minutes...
   An unexpected error has occurred: CalledProcessError: command: ('/Users/bjornpedersen/.pyenv/versions/3.10.5/envs/env105/bin/python3.10', '-mvirtualenv', '/Users/bjornpedersen/.cache/pre-commit/repoj99o3uv5/py_env-python3.7', '-p', 'python3.7')
   return code: 1
   expected return code: 0
   stdout:
       RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'
       
   stderr: (none)
   Check the log at /Users/bjornpedersen/.cache/pre-commit/pre-commit.log
   ```
   
   ### Issue Priority
   
   Priority: 2
   
   ### Issue Component
   
   Component: sdk-py-core


-- 
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.apache.org

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

[GitHub] [beam] damccorm closed issue #24312: Update Python version in .pre-commit-config.yaml

Posted by GitBox <gi...@apache.org>.
damccorm closed issue #24312: Update Python version in .pre-commit-config.yaml
URL: https://github.com/apache/beam/issues/24312


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

[GitHub] [beam] BjornPrime commented on issue #24312: Update Python version in .pre-commit-config.yaml

Posted by GitBox <gi...@apache.org>.
BjornPrime commented on issue #24312:
URL: https://github.com/apache/beam/issues/24312#issuecomment-1323884771

   .take-issue


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

[GitHub] [beam] BjornPrime commented on issue #24312: Update Python version in .pre-commit-config.yaml

Posted by GitBox <gi...@apache.org>.
BjornPrime commented on issue #24312:
URL: https://github.com/apache/beam/issues/24312#issuecomment-1323885849

   I propose we loosen the restriction to allow all Python 3 environments to run pre-commits. If we want more specificity than that for some reason, I believe we can add multiple versions to the list, so we can just keep that updated to include all currently supported versions of python3, though this would require ongoing effort and I'm unclear on whether it would have any advantages.


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