Skip to content

Conversation

@thomasjpfan
Copy link
Contributor

@thomasjpfan thomasjpfan commented Oct 31, 2025

Describe your changes

I noticed that caching is not working for one of the MacOS runners. This PR changes the python setup code to use uv instead to see if it's better.

Checklists

Compatibility checklist

Check these boxes or delete any item (or this section) if not relevant for this PR.

  • Client+Server: this change is compatible with old servers
  • Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.


Release checklist

If you intend for this commit to trigger a full release to PyPI, please ensure that the following steps have been taken:

  • Version file (modal_version/__init__.py) has been updated with the next logical version
  • Changelog has been cleaned up and given an appropriate subhead


Note

Switches the composite GitHub Action to install and cache Python dependencies with uv instead of pip/actions/cache.

  • CI / GitHub Actions:
    • Replace actions/cache with astral-sh/setup-uv to enable dependency caching via uv (enable-cache: true, cache-dependency-glob for requirements.dev.txt and pyproject.toml).
    • Install dependencies with uv pip install -r requirements.dev.txt and set UV_SYSTEM_PYTHON=1 (remove manual pip upgrade/install).
    • Keep Python setup via actions/setup-python unchanged; pin uv version to 0.9.7.

Written by Cursor Bugbot for commit 2014560. This will update automatically on new commits. Configure here.

@thomasjpfan thomasjpfan changed the title Use uv to install dependencies Use uv to install python dependencies in tests Oct 31, 2025
@thomasjpfan thomasjpfan marked this pull request as ready for review November 1, 2025 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants