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

Update: isort pre-commit hook #120

Merged
merged 1 commit into from Apr 6, 2023

Conversation

mikemckiernan
Copy link
Contributor

Running pre-commit run --all-files on
ref 6df4751, the current main branch,
fails to complete and reports the foll in
the error:

[extras.pipfile_deprecated_finder.2] 'pip-shims<=0.3.4' does not match '^[a-zA-Z-_.0-9]+$'

This seems to be the same error as PyCQA/isort#2077.

I'm proposing that we update to the same isort
that the isort repo currently uses.

Running `pre-commit run --all-files` on
ref `6df4751`, the current `main` branch,
fails to complete and reports the foll in
the error:

[extras.pipfile_deprecated_finder.2] 'pip-shims<=0.3.4' does not match '^[a-zA-Z-_.0-9]+$'

I _think_ this is related to PyCQA/isort#2079.

I'm proposing that we update to the same isort
that the isort repo currently uses.
Copy link
Member

@choldgraf choldgraf left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for updating it. For some reason I don't see the "approve to run github actions" button, but I think it's probably fine given that this is just a pre-commit hook and it's broken right now

@choldgraf choldgraf changed the title fix: isort hook fails Update: isort pre-commit hook Apr 6, 2023
@choldgraf choldgraf merged commit c471d0e into executablebooks:main Apr 6, 2023
@welcome
Copy link

welcome bot commented Apr 6, 2023

Congrats on your first merged pull request in this project! 🎉
congrats

Thank you for contributing, we are very proud of you! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants