Sebastian Rittau comments

Results 518 comments of


                                            Sebastian Rittau

[stubsabot] Bump Authlib to 1.6.0

Closing to let stubsabot generate a new PR.

Fix #13881: Add plugin to disallow bytearray as filename in compile()

Please note that typeshed only provides type annotations, we don't need or want any custom mypy plugins – that would be detrimental to typeshed's purpose. To solve #13881, we just...

colorful how to handle dynamic types

If I understand how colorful works correctly, I fear the only possible solution is to basically copy the whole API of `Colorful` into `colorful/__init__.pyi`, including `__getattr__`, which means that indeed...

2 different `Dialect` types - one in `csv` module and other in `_csv` module

This is a bit unfortunate, mostly because the implementation in Python is a bit hacky. We could in theory just declare `csv.Dialect` to be an alias of `_csv.Dialect` (or a...

fix `list.add` to use the expected type

This has the same problem as #14755: `_T1` has the same meaning as `Any`, so this is what we should use (with an appropriate comment). Also, the tests are not...

fix `collections.abc.Callable` and `typing.Callable`

I'm always in favor of bringing our stubs more in line with reality, so in general I'm in favor of the change. But unfortunately at least mypy seems heavily dependent...

fix `collections.abc.Callable` and `typing.Callable`

> this is a circular dependency. how can we ever change anything if the change needs to be compatible with our dependants In that case this needs coordination with the...

fix `collections.abc.Callable` and `typing.Callable`

Yes, @erictraut might be able to help.

TaskGroup create_task missing eager_start and kwargs

Added in 3.14. I'm not sure we should add `kwargs`, though. Instead copying the actual args from `Task.__init__()` seems safest to me. (Although as I'm not a great asyncio expert,...

Constrain memoryview type var to allowed types

I was thinking about that, but I think changing the default to `Any` might be a better solution. That keeps the flexibility of restricting memoryviews while by default any memoryview...