BumpDeps: mypy #2545
BumpDeps: mypy #2545
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
github-actions
bot
requested review from
squirrelsc and
LiliDeng
as code owners
|
Brought in the typing changes from the unpinning PR. This should be ready to go. |
|
@LiliDeng When you have time, please let Avram know which scenarios should be covered. For the marketplace image, SIG loading tests, it may need to run multiple test cases together, which will load image object from memory. |
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
from
fa09b81
to
2d11181
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
2 times, most recently
from
2cbde7e
to
6efe1d7
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
2 times, most recently
from
6af232c
to
0238a7d
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
from
0238a7d
to
1019849
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
from
1019849
to
de8c44f
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
from
de8c44f
to
5d6686b
Compare
avylove
force-pushed
the
bumpdeps/mypy_3982549804
branch
from
5d6686b
to
d2befb6
Compare
|
@squirrelsc Rebased again. This basic maintenance PR is now 3 months old. and we're running a version of Mypy that was superseded a year ago. |
|
It contains many changes, we don't have bandwidth to test regressions. Please disable related rules, and focus on upgrading mypy version in this PR. |
|
That's what I am doing. These are just the changes to make it pass. It keeps growing because it hasn't been merged in so long and bad code keep getting implemented. This is why you have to keep up with dependencies. |
|
Please split changes into separated PRs, and provide tested scenarios. We can evaluate the test coverage and remaining risks. |
|
What is there to split? These are the changes need to work with the new version. This is a relatively simple PR for anyone with intermediate Python skills to review. |
Created by Github Action