Skip to content

[pre-commit.ci] pre-commit autoupdate#2045

Closed
pre-commit-ci[bot] wants to merge 2 commits intomasterfrom
pre-commit-ci-update-config
Closed

[pre-commit.ci] pre-commit autoupdate#2045
pre-commit-ci[bot] wants to merge 2 commits intomasterfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Sep 13, 2021

Comment on lines -6 to -14

from shutil import which
# we try our best, but the shebang of this script is difficult to determine:
# - macos doesn't ship with python3
# - windows executables are almost always `python.exe`
# therefore we continue to support python2 for this small script
if sys.version_info < (3, 3):
from distutils.spawn import find_executable as which
else:
from shutil import which
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm -- will need to make a conscious decision as to whether to drop this or not 🤔

updates:
- [github.com/asottile/pyupgrade: v2.25.0 → v2.28.0](asottile/pyupgrade@v2.25.0...v2.28.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 8962330 to 02169ee Compare September 27, 2021 18:49
@asottile asottile deleted the pre-commit-ci-update-config branch September 28, 2021 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant