Optimize CI by caching uv virtual environment and speed the testing#610
Closed
archit7-beep wants to merge 1 commit intomllam:mainfrom
Closed
Optimize CI by caching uv virtual environment and speed the testing#610archit7-beep wants to merge 1 commit intomllam:mainfrom
archit7-beep wants to merge 1 commit intomllam:mainfrom
Conversation
Contributor
Collaborator
|
Hi, I don't think we need a second PR here, instead you could add your review directly to #606 |
21 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your changes
This PR improves the robustness of the CI caching mechanism for uv-based virtual environments. It ensures that the cached .venv is safely reused on cache hits and avoids potential failures caused by stale or partially restored environments.
The current workflow assumes that a restored .venv is always valid. However, in rare cases (e.g., cache corruption or incomplete restoration), this can lead to runtime failures.
This change introduces a safer handling of cached environments by ensuring that the workflow behaves correctly across both cache hits and misses, without altering the existing dependency installation logic or CI behavior.
No new dependencies are introduced. The changes rely only on existing tools (uv, pip, and GitHub Actions caching).
Issue Link
Closes (#605 )
Type of change
Checklist before requesting a review
pullwith--rebaseoption if possible).Checklist for reviewers
Each PR comes with its own improvements and flaws. The reviewer should check the following:
Author checklist after completed review
reflecting type of change (add section where missing):
Checklist for assignee