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 importlib_resources as it breaks pytest_rewrites #38095

Merged
merged 1 commit into from Mar 13, 2024

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Mar 12, 2024

New version of importlib_resources released today - 6.2.0 but also 6.3.0 released few hours later, break pytest_rewrite feature in rather unexpected ways. Until the problem is diagnosed and solved we should limit it. The issue is tracked in
python/importlib_resources#299


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

potiuk commented Mar 12, 2024

also fixes a typo in skipping images jobs (but that's secondary)

@potiuk potiuk requested a review from eladkal March 13, 2024 00:04
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299
@potiuk potiuk requested a review from kaxil as a code owner March 13, 2024 01:50
@potiuk potiuk force-pushed the limit-importlib-resources branch 3 times, most recently from 3aec930 to 4f265ec Compare March 13, 2024 01:51
dev/breeze/pyproject.toml Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
@potiuk potiuk merged commit 7ec2407 into apache:main Mar 13, 2024
95 of 100 checks passed
@potiuk potiuk deleted the limit-importlib-resources branch March 13, 2024 08:05
drajguru pushed a commit to drajguru/airflow that referenced this pull request Mar 14, 2024
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Mar 14, 2024
howardyoo pushed a commit to howardyoo/airflow that referenced this pull request Mar 18, 2024
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299
potiuk added a commit that referenced this pull request Mar 19, 2024
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299

(cherry picked from commit 7ec2407)
@jedcunningham jedcunningham added this to the Airflow 2.8.4 milestone Mar 19, 2024
@jedcunningham jedcunningham removed the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Mar 19, 2024
@jedcunningham jedcunningham added the type:misc/internal Changelog: Misc changes that should appear in change log label Mar 19, 2024
howardyoo pushed a commit to howardyoo/airflow that referenced this pull request Mar 31, 2024
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299
utkarsharma2 pushed a commit to astronomer/airflow that referenced this pull request Apr 22, 2024
New version of importlib_resources released today - 6.2.0 but also 6.3.0
released few hours later, break pytest_rewrite feature in rather
unexpected ways. Until the problem is diagnosed and solved we should
limit it. The issue is tracked in
python/importlib_resources#299
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.

None yet

6 participants