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

chore: Update uv version to latest #5086

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

ntkathole
Copy link
Contributor

What this PR does / why we need it:

This PR updates the uv version to latest. Using old version of uv, it was removing required dependencies while compiling the pip.

New version is ensuring more accurate resolution of package versions and requirements, also, prevents unnecessary removals by better handling version conflicts and sub-dependencies.

Which issue(s) this PR fixes:

This was blocking CI to pass while trying to add new add on.

@ntkathole ntkathole requested a review from a team as a code owner February 25, 2025 06:57
@ntkathole ntkathole force-pushed the update_uv branch 3 times, most recently from 992abe6 to 1af4a70 Compare February 25, 2025 09:48
@@ -70,8 +70,8 @@ gcp = [
"google-cloud-bigtable>=2.11.0,<3",
"fsspec<=2024.9.0",
]
ge = ["great_expectations>=0.15.41"]
go = ["cffi~=1.15.0"]
ge = ["great_expectations>=0.15.41,<1"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

temp_module = importlib.import_module(full_name)
if is_pkg:
next_packages.append(temp_module)
if "ikv" not in full_name and "milvus" not in full_name:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ge = ["great_expectations>=0.15.41"]
go = ["cffi~=1.15.0"]
ge = ["great_expectations>=0.15.41,<1"]
go = ["cffi>=1.15.0"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@franciscojavierarceo franciscojavierarceo enabled auto-merge (squash) February 25, 2025 11:12
@franciscojavierarceo franciscojavierarceo merged commit e25ca4f into feast-dev:master Feb 25, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants