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

Limit temporarily aiohttp to < 3.11.0 #43983

Closed
wants to merge 1 commit into from

Conversation

vincbeck
Copy link
Contributor

@vincbeck vincbeck commented Nov 13, 2024

Because of aio-libs/aiohttp#9866 some tests are failing. This PR can be reverted when pnuckowski/aioresponses#262 is merged and released


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@potiuk
Copy link
Member

potiuk commented Nov 13, 2024

This PR (when you open failing output you can see it) revealed limitation on parsing development dependencies that should be installed for compatiblity testing.

The fix is here: #43995 - but I think this limit should not be placed in devel-deps. I will take a look in a moment.

@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

This is a better fix #43998 - aiohttp is already a dependency of adlfs so we can add it there.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better fix in #43998

@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

Even better fix #44006

@potiuk
Copy link
Member

potiuk commented Nov 14, 2024

Closing -> the fix from #44006 is much more "appropriate".

@potiuk potiuk closed this Nov 14, 2024
@vincbeck vincbeck deleted the vincbeck/aiohttp branch November 18, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants