-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Bump problematic dependencies to avoid backtracking #51702
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
Conversation
6a0995e to
de0a6b2
Compare
jscheffl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's see!
|
What is your approach for such selection? Taking the package with the most number of releases since the lower pin you find in the repo? |
Pure guessing. .. Maybe @notatallshaw might have some better hints? FYI @notatallshaw -> we again had problems with backtracking with |
|
Added pypa/pip#13281 (comment) -> maybe Damian will be able to help again. |
|
FYI I can reproduce it and am taking a look, no promises, but if I can find a solution I will open a PR. |
|
🙇 🙇 |
|
Btw, I've also merged sarugaku/resolvelib#188 which I hope to get in to pip 25.2. This will restore the biggest optimization, for most cases, that was available in 25.0 and lost in 25.1. 25.1 also added several other optimizations, and I am experimenting with an additional optimization which I think will particularly benefit Airflow's CI, so I'm hopeful 25.2 will be a lot less problematic for you. |
^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.