anthony sottile
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: ... ```
looks like #276 is where the issue was introduced
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...