Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/update pre commit config #443

Merged
merged 2 commits into from
Sep 10, 2021

Conversation

TomAugspurger
Copy link
Member

This updates the pre-commit config to match Dask's (minus isort, which wasn't previously used here).

This fixes an issue I saw locally with the "language_version" and a conda env:

An unexpected error has occurred: CalledProcessError: command: ('/home/taugspurger/miniconda3/envs/dask-gateway/bin/python3.8', '-mvirtualenv', '/home/taugspurger/.cache/pre-commit/repoi452dik9/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'

This updates the pre-commit config to match Dask's (minus isort, which
wasn't used previously. We can add that later if we want).
@jacobtomlinson jacobtomlinson merged commit 7ef6b7a into dask:main Sep 10, 2021
@TomAugspurger TomAugspurger deleted the fix/update-pre-commit-config branch September 10, 2021 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants