On Mon, Mar 1, 2021 at 2:38 AM Hynek Schlawack <hs@ox.cx> wrote:
FWIW, I think it’s common (aka _I_ do it that way) that .post are indeed post-releases like a documentation update that doesn’t warrant a proper update. PEP 440 specifies it and RTD has good support for that. E.g. https://www.attrs.org/en/17.3.0.post2/
IIRC Twisted doesn’t bump before the release, which makes this whole workflow unfeasible? But it’s something to keep in mind when deciding.
I had some questions about PEP440, so I asked for feedback from Łukasz Langa who provided it here: https://github.com/twisted/twisted/pull/1514#issuecomment-788059407 Basically Łukasz's feedback was that doing this: 21.2.0.rc1 21.2.0 (this is the release) 21.2.0.post0 is OK. So in PR 1514, I went with that. Unfortunately, I ran into a bug in Twisted incremental: File "/tmp/pip-req-build-zvf8johl/src/twisted/__init__.py", line 11, in <module> from twisted._version import __version__ as version File "/tmp/pip-req-build-zvf8johl/src/twisted/_version.py", line 10, in <module> __version__ = Version("Twisted", 21, 2, 0, post=0) TypeError: __init__() got an unexpected keyword argument 'post' I might need some help to get a fix into incremental and push that out later today, so I can complete PR 1514. -- Craig