ci: add uv virtual environment cache to setup-uv-project action#559
ci: add uv virtual environment cache to setup-uv-project action#559davidberenstein1957 wants to merge 3 commits intomainfrom
Conversation
Made-with: Cursor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
| uses: actions/cache@v5 | ||
| with: | ||
| path: .venv | ||
| key: uv-venv-${{ runner.os }}-${{ matrix.python-version || '3.11' }}-${{ hashFiles('uv.lock', 'pyproject.toml') }} |
There was a problem hiding this comment.
Cache key references gitignored uv.lock file
Medium Severity
The uv.lock file is listed in .gitignore, so it will never be present in the repository checkout. hashFiles('uv.lock', 'pyproject.toml') silently ignores missing files, so the cache key effectively only hashes pyproject.toml. This means the uv.lock reference is a no-op, and the cache won't invalidate when it otherwise would if the lock file were tracked. The same issue applies to the restore-keys line.
Made-with: Cursor
Made-with: Cursor


Description
Added some caching for UV :)
Related Issue
Fixes #(issue number)
Type of Change
How Has This Been Tested?
Checklist
Additional Notes