Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v1.8 failed to release #698

Closed
1 task done
mjpieters opened this issue Mar 28, 2022 · 5 comments
Closed
1 task done

v1.8 failed to release #698

mjpieters opened this issue Mar 28, 2022 · 5 comments
Labels

Comments

@mjpieters
Copy link
Contributor

mjpieters commented Mar 28, 2022

Describe the bug

The 1.8 tag exists, but the release step failed:

Traceback (most recent call last):
  File "/get_releasenote.py", line 256, in <module>
    sys.exit(main())
  File "/get_releasenote.py", line 229, in main
    ctx.version = find_version(
  File "/get_releasenote.py", line 206, in find_version
    raise ValueError(f"Unable to determine version in file '{version_file}'")
ValueError: Unable to determine version in file 'yarl/__init__.py'

As a result, the most recent version on PyPI is still 1.7.2, which doesn't have the new .suffix support. The stable tagged RTD documentation does include URL.suffix, which caused me some confusion.

I'd love to have exactly that feature for a project I'm contributing to, can the release be pushed out, even if only manually?

yarl Version

$ python -m pip show yarl
Name: yarl
Version: 1.7.2
Summary: Yet another URL library
Home-page: https://github.com/aio-libs/yarl/
Author: Andrew Svetlov
Author-email: andrew.svetlov@gmail.com
License: Apache 2
Location: /Users/mj/Library/Caches/pypoetry/virtualenvs/nitpick-ImfW6xDq-py3.7/lib/python3.7/site-packages
Requires: idna, multidict, typing-extensions
Required-by:

OS

macOS

Code of Conduct

  • I agree to follow the aio-libs Code of Conduct
@mjpieters mjpieters added the bug label Mar 28, 2022
@mjpieters
Copy link
Contributor Author

It looks like a potential fix is in the works.

@asvetlov what’s the status there? Anything I can do that would unblock this?

@asvetlov
Copy link
Member

Heh.
Could you help with get_releasenote.py fixing?
Sorry, I'm spending my spare time on CPython/asyncio improvements.
Python 3.11 feature freeze is in a month.

@mjpieters
Copy link
Contributor Author

mjpieters commented Apr 1, 2022

Could you help with get_releasenote.py fixing?

I'll see what I can do. At first glance your regex can't work (\[number] references don't work inside character classes), that's probably the root cause. See the re docs:

Inside the '[' and ']' of a character class, all numeric escapes are treated as characters.

Use a negative look ahead instead: https://stackoverflow.com/q/8055727

@mjpieters
Copy link
Contributor Author

Apart from the get-releasenote issue, there is another: the 1.8 tag doesn't actually contain a 1.8 version string. We'll need to remedy that first!

What is the proper release process here?

  • Update the version string
  • Move the 1.8 tag, or, if that's preferred instead, tag 1.8.1.

Anything else?

@webknjaz
Copy link
Member

I'm going to close this because the action items mentioned here are mostly complete, please read further investigation and steps @ #706 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants