typeshed icon indicating copy to clipboard operation
typeshed copied to clipboard

Rework redis/asyncio/{connection,sentinel}.pyi

Open srittau opened this issue 2 years ago • 6 comments

Closes: #10362

srittau avatar Jun 27 '23 21:06 srittau

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

github-actions[bot] avatar Jun 27 '23 21:06 github-actions[bot]

I have no time to look into the pytype error at the moment. @rchen152: Could have a look? Otherwise I'll try to narrow the problem down tomorrow.

srittau avatar Jun 27 '23 21:06 srittau

The pytype fix was fairly straightforward: Remove the quotes around bound types from the TypeVar, which is cleaner anyway:

https://github.com/python/typeshed/pull/10370/commits/45482b5cecfc71cae0766de9b5382bcf9b46e091

srittau avatar Jun 28 '23 10:06 srittau

(Test failure is a GitHub hickup, I will restart the failed test.)

srittau avatar Jun 28 '23 10:06 srittau

Diff from mypy_primer, showing the effect of this PR on open source code:

vision (https://github.com/pytorch/vision): typechecking got 2.80x faster (19.1s -> 6.8s)
(Performance measurements are based on a single noisy sample)

manticore (https://github.com/trailofbits/manticore): typechecking got 2.42x faster (44.9s -> 18.6s)
(Performance measurements are based on a single noisy sample)

pandas (https://github.com/pandas-dev/pandas): typechecking got 2.59x faster (234.8s -> 90.5s)
(Performance measurements are based on a single noisy sample)

github-actions[bot] avatar Jun 28 '23 10:06 github-actions[bot]

Sorry this PR has been sitting here for so long. There is a fairly big merge conflict; if you fix that I'll review it.

JelleZijlstra avatar Feb 05 '24 13:02 JelleZijlstra

@srittau ping on this one.

JelleZijlstra avatar Mar 15 '24 05:03 JelleZijlstra

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

github-actions[bot] avatar Mar 15 '24 11:03 github-actions[bot]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

github-actions[bot] avatar Mar 15 '24 11:03 github-actions[bot]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

github-actions[bot] avatar Mar 15 '24 11:03 github-actions[bot]