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

Bump uv to 0.5.14 #45394

Merged
merged 2 commits into from
Jan 4, 2025
Merged

Bump uv to 0.5.14 #45394

merged 2 commits into from
Jan 4, 2025

Conversation

gopidesupavan
Copy link
Member


^ 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 Jan 4, 2025

One comment. I think we have one or two places where uv is not bumped automatically by precommit (I marked them with TODO(potiuk) - maybe worth finding it now and updating/reapplying the pre-commit?

@gopidesupavan
Copy link
Member Author

One comment. I think we have one or two places where uv is not bumped automatically by precommit (I marked them with TODO(potiuk) - maybe worth finding it now and updating/reapplying the pre-commit?

yeah good call :) updated

@potiuk
Copy link
Member

potiuk commented Jan 4, 2025

Shall update automation for that one too (i.e. update pre-commit for the next time ) ? I can do it, separately, no problem as well :). I think it's the right time.

@potiuk
Copy link
Member

potiuk commented Jan 4, 2025

Looks like we have the first serious case where new uv version behaves differently - looks like resolving the dependencies for lowest-direct is failing with some pandas problem. I will re-run to check, but It's likely we will have to adapt something.

@gopidesupavan
Copy link
Member Author

Looks like we have the first serious case where new uv version behaves differently - looks like resolving the dependencies for lowest-direct is failing with some pandas problem. I will re-run to check, but It's likely we will have to adapt something.

Yeah interesting ...

@gopidesupavan
Copy link
Member Author

Looks like we have the first serious case where new uv version behaves differently - looks like resolving the dependencies for lowest-direct is failing with some pandas problem. I will re-run to check, but It's likely we will have to adapt something.

Thanks for updating :)

@potiuk potiuk force-pushed the bump-uv-to-0.5.14 branch from 24640d4 to b312c8d Compare January 4, 2025 19:10
@potiuk
Copy link
Member

potiuk commented Jan 4, 2025

accidental issue only :)

@potiuk potiuk merged commit 801b92e into apache:main Jan 4, 2025
155 of 156 checks passed
LefterisXefteris pushed a commit to LefterisXefteris/airflow that referenced this pull request Jan 5, 2025
agupta01 pushed a commit to agupta01/airflow that referenced this pull request Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-tools area:production-image Production image improvements and fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants