Skip to content

chore: Update uv version to latest - #5086

Merged
franciscojavierarceo merged 1 commit into
feast-dev:masterfrom
ntkathole:update_uv
Feb 25, 2025
Merged

franciscojavierarceo merged 1 commit into
feast-dev:masterfrom
ntkathole:update_uv

Conversation

@ntkathole

Copy link
Copy Markdown
Member

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
Comment thread pyproject.toml
]
ge = ["great_expectations>=0.15.41"]
go = ["cffi~=1.15.0"]
ge = ["great_expectations>=0.15.41,<1"]

Copy link
Copy Markdown
Member 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
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread pyproject.toml
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
Copy Markdown
Member 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
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