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

Lift the limit on importlib_resources library #38294

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Mar 19, 2024

The problem with importlib_resources interacting badly with pytest_rewrite has been solved in 6.3.2 version today and we can simply skip over the bad versions.

See python/importlib_resources#298


^ 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.

The problem with importlib_resources interacting badly with
pytest_rewrite has been solved in 6.3.2 version today and we can
simply skip over the bad versions.

See python/importlib_resources#298
@potiuk potiuk merged commit 2cb4209 into apache:main Mar 19, 2024
88 checks passed
@potiuk potiuk deleted the lift-importlib-resources-limt branch March 19, 2024 20:57
@jedcunningham jedcunningham added this to the Airflow 2.8.4 milestone Mar 19, 2024
@jedcunningham jedcunningham added the type:misc/internal Changelog: Misc changes that should appear in change log label Mar 19, 2024
jedcunningham pushed a commit that referenced this pull request Mar 19, 2024
The problem with importlib_resources interacting badly with
pytest_rewrite has been solved in 6.3.2 version today and we can
simply skip over the bad versions.

See python/importlib_resources#298

(cherry picked from commit 2cb4209)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-tools type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants