GitHub Actions: Installing frozenlist fails on python3.11-beta.5

[cclauss]

An on-demand GitHub Action...

name: python_3_11-dev
on:
  workflow_dispatch:
jobs:
  python_3_11-dev:
    strategy:
      fail-fast: false
      matrix:
        package:
          - cchardet  # Passes if cython>=0.29.30
          - frozenlist  # Fails
          - yarl  # Fails
          - git+https://github.com/PyYoshi/cChardet.git  # Passes!!!
          - git+https://github.com/aio-libs/frozenlist.git  # Fails
          - git+https://github.com/aio-libs/yarl.git  # Fails
    runs-on: ubuntu-latest
    steps:
      - uses: actions/setup-python@v4
        with:
          python-version: 3.11-dev
      - run: python3.11 -m pip install --upgrade pip wheel
      - run: python3.11 -m pip install --upgrade Cython>=0.29.30
      - run: python3.11 -m pip install ${{ matrix.package }}

Test a workaround for #6600

• https://github.com/aio-libs/aiohttp/pull/6830/files
• https://github.com/aio-libs/aiohttp/runs/7379083164?check_suite_focus=true

aio-libs/yarl#706 (comment)

What do these changes do?

Are there changes in behavior for the user?

Related issue number

Checklist

• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes
• If you provide code modification, please add yourself to CONTRIBUTORS.txt
  • The format is <Name> <Surname>.
  • Please keep alphabetical order, the file is sorted by names.
• Add a new news fragment into the CHANGES folder
  • name it <issue_id>.<type> for example (588.bugfix)
  • if you don't have an issue_id change it to the pr id after creating the pr
  • ensure type is one of the following:
    • .feature: Signifying a new feature.
    • .bugfix: Signifying a bug fix.
    • .doc: Signifying a documentation improvement.
    • .removal: Signifying a deprecation or removal of public API.
    • .misc: A ticket has been closed, but it is not of interest to users.
  • Make sure to use full sentences with correct case and punctuation, for example: "Fix issue with non-ascii contents in doctest text files."

[codecov]

Codecov Report

Merging #6830 (836c5ff) into master (0db3362) will increase coverage by 71.67%.
The diff coverage is n/a.

❗ Current head 836c5ff differs from pull request most recent head f214f13. Consider uploading reports for the commit f214f13 to get more accurate results

@@             Coverage Diff             @@
##           master    #6830       +/-   ##
===========================================
+ Coverage   21.76%   93.44%   +71.67%     
===========================================
  Files         104      104               
  Lines       30628    30627        -1     
  Branches     3064     3076       +12     
===========================================
+ Hits         6666    28619    +21953     
+ Misses      23859     1838    -22021     
- Partials      103      170       +67     

Flag	Coverage Δ
unit	93.36% <ø> (+71.62%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
aiohttp/client_ws.py	21.96% <0.00%> (ø)
tests/autobahn/test_autobahn.py	38.33% <0.00%> (+6.66%)	⬆️
tests/test_circular_imports.py	100.00% <0.00%> (+7.69%)	⬆️
aiohttp/abc.py	100.00% <0.00%> (+7.86%)	⬆️
aiohttp/web_exceptions.py	99.52% <0.00%> (+22.27%)	⬆️
tests/conftest.py	68.42% <0.00%> (+25.43%)	⬆️
aiohttp/resolver.py	64.00% <0.00%> (+28.00%)	⬆️
aiohttp/tracing.py	100.00% <0.00%> (+36.12%)	⬆️
tests/test_resolver.py	70.07% <0.00%> (+37.79%)	⬆️
aiohttp/client_exceptions.py	96.66% <0.00%> (+39.16%)	⬆️
... and 85 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[Dreamsorcerer]

You might want to revert the mypy version change, I'll fix the new errors in a separate PR.

[owocado]

I think the original post might need to be updated to mention that pip install of latest version of yarl (>=1.8.1) passes with success on latest python v3.11-dev (3.11.0.beta5) since the OP seems outdated due to # Fails comments on 2 yarl entries.

[cclauss]

@ow0x You are correct.
cython, frozenlist, and yarl have all been updated and release since the original commit message.
The problem now seems to be apipkg which is tested on Py3.11 but seems to fail here.

[cclauss]

Sorry. It will take someone with steadier hands than mine to make this green on Py3.11. I sense that master is not even the correct branch to work on.

[Dreamsorcerer]

We'll want master working and then the 3.9 branch too (and maybe 3.8 if it's not much extra work).

[Dreamsorcerer]

I think the apipkg error is caused by an outdated pytest (judging by the error reports pytest-dev/apipkg#30, pytest-dev/py#273, pytest-dev/pytest#9181).

[webknjaz]

I think the apipkg error is caused by an outdated pytest

Yes, confirmed by upgrading it.