Skip to content

Conversation

srittau
Copy link
Collaborator

@srittau srittau commented Jan 3, 2022

stubgen + import cleanup + mypy fixes + black + isort

@srittau
Copy link
Collaborator Author

srittau commented Jan 3, 2022

I'm out of ideas.

@JelleZijlstra
Copy link
Member

We'll probably need to change stubtest to try-except more exception types in this case. Unfortunately that may mean we have to wait for the next mypy release.

JelleZijlstra added a commit to python/mypy that referenced this pull request Jan 9, 2022
@srittau srittau added the status: deferred Issue or PR deferred until some precondition is fixed label Jan 9, 2022
@srittau
Copy link
Collaborator Author

srittau commented Jan 9, 2022

I'm marking this as deferred for now.

@hauntsaninja
Copy link
Collaborator

hauntsaninja commented Jan 12, 2022

There's a list of distributions to skip running stubtest on; you could add to that to unblock this PR.

Edit: looks like this was changed to a TOML flag in #6687

JelleZijlstra added a commit to python/mypy that referenced this pull request Jan 12, 2022
@srittau
Copy link
Collaborator Author

srittau commented Jan 12, 2022

The toml flag was removed in the meantime as well as we fixed all remaining problems. We could reintroduce it, but we can also just wait for the next mypy release.

tushar-deepsource pushed a commit to DeepSourceCorp/mypy that referenced this pull request Jan 20, 2022
@srittau srittau mentioned this pull request Jan 24, 2022
@srittau
Copy link
Collaborator Author

srittau commented Feb 22, 2022

#7351 will allow us to merge despite stubtest.

@srittau srittau removed the status: deferred Issue or PR deferred until some precondition is fixed label Feb 22, 2022
@JelleZijlstra JelleZijlstra merged commit 7e5d2a5 into python:master Mar 7, 2022
@srittau srittau deleted the openpyxl branch March 7, 2022 07:23
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.

3 participants