Skip to content

chore: fix mypy pre-commit hook - #1562

Merged
JohnVillalovos merged 3 commits into
masterfrom
chore/fix-mypy-pre-commit
Aug 23, 2021
Merged

chore: fix mypy pre-commit hook#1562
JohnVillalovos merged 3 commits into
masterfrom
chore/fix-mypy-pre-commit

Conversation

@nejch

@nejch nejch commented Aug 1, 2021

Copy link
Copy Markdown
Member

Closes #1539.

Additionally, since mypy 0.910 doesn't bundle stubs anymore we need to track them for pre-commit's own virtualenv :/ See pre-commit/pre-commit#730

@nejch
nejch marked this pull request as draft August 1, 2021 12:29
@nejch
nejch force-pushed the chore/fix-mypy-pre-commit branch from 7db1482 to 430c332 Compare August 1, 2021 12:31
@nejch
nejch marked this pull request as ready for review August 1, 2021 19:00
@nejch
nejch requested a review from JohnVillalovos August 1, 2021 19:00
Comment thread pyproject.toml
Comment thread pyproject.toml
Comment thread tox.ini Outdated
@javatarz

Copy link
Copy Markdown
Contributor

Can we please get this merged? The pre-commit hooks are broken and the mypy stage blocks commits every time.

The workaround I've been using is to disable mypy locally (via a commit) and then remove the commit before pushing. This is painful.

@nejch

nejch commented Aug 22, 2021

Copy link
Copy Markdown
Member Author

Can we please get this merged? The pre-commit hooks are broken and the mypy stage blocks commits every time.

The workaround I've been using is to disable mypy locally (via a commit) and then remove the commit before pushing. This is painful.

Sorry, I know this was really annoying!

@JohnVillalovos I'm back from the dead 😆 would you be able to take a quick look/merge?

@nejch
nejch requested a review from JohnVillalovos August 22, 2021 16:05
@JohnVillalovos
JohnVillalovos merged commit 7a64e67 into master Aug 23, 2021
@JohnVillalovos
JohnVillalovos deleted the chore/fix-mypy-pre-commit branch August 23, 2021 05:25
@JohnVillalovos

Copy link
Copy Markdown
Member

Thanks @nejch !

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.

pre-commit hook for mypy fails

4 participants