Skip to content

Update isort version in pre-commit config #36428

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

Merged
merged 1 commit into from
Sep 17, 2020

Conversation

clbarnes
Copy link
Contributor

@clbarnes clbarnes commented Sep 17, 2020

Previously, a mismatch between the isort versions specified by
pre-commit and by the files used on CI made it impossible to make a
commit which passed both pre-commit and CI lints.

See #36426 . This is not a complete solution as the flake8 version differences are intentional and hopefully temporary (see #36412 (comment) ), but it should resolve the current breakage.

Previously, a mismatch between the isort versions specified by
pre-commit and by the files used on CI made it impossible to make a
commit which passed both pre-commit and CI lints.
@jreback jreback added the Code Style Code style, linting, code_checks label Sep 17, 2020
@jreback jreback added this to the 1.2 milestone Sep 17, 2020
@jreback jreback merged commit 81d24c7 into pandas-dev:master Sep 17, 2020
@jreback
Copy link
Contributor

jreback commented Sep 17, 2020

thanks @clbarnes

rhshadrach pushed a commit to rhshadrach/pandas that referenced this pull request Sep 17, 2020
Previously, a mismatch between the isort versions specified by
pre-commit and by the files used on CI made it impossible to make a
commit which passed both pre-commit and CI lints.
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
Previously, a mismatch between the isort versions specified by
pre-commit and by the files used on CI made it impossible to make a
commit which passed both pre-commit and CI lints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Style Code style, linting, code_checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants