anthony sottile

Results 159 comments of anthony sottile

either of those, yes -- I actually don't think the pypi package is even usable on its own right now 😆

in the above example the "hi there" is the yank message -- afaict there's no limit on it: https://github.com/pypi/warehouse/blob/a3d6221d4c692569e2290f6d75a75b4db747335b/warehouse/packaging/models.py#L468

I'm not sure we'll have much control over the newlines and whitespace -- and we only get one message for the warning case but otherwise looks good

we get an error message per package so we can use the right one for each

oh I want to set the yank messages now so that we're ready to turn it off for the next calver release :D

yeah the message will be entered after that -- yanking is entirely reversible which is why we're approaching from that angle rather than deleting. the message can be changed yes...

fwiw `reorder-python-imports` seems to get this right: ```diff $ reorder-python-imports --add-import 'from __future__ import annotations' t.py Reordering imports in t.py $ diff -u t.py{.bak,} --- t.py.bak 2022-09-14 11:38:00.148863504 -0400 +++...

as a workaround: ```python def docker_from_env() -> ContextManager[docker.DockerClient]: return contextlib.closing(docker.from_env()) ``` usage: ```python with docker_from_env() as client: ... ```

there isn't a way to reliably do that because the session is in a thread local (and thus leaks **once per thread**) plus relying on any user of your library...