Skip to content

Fix setup-uv cache dependency config in lint and test workflows#53

Open
smortezah wants to merge 5 commits intotoon-format:mainfrom
smortezah:fix/uv-cache
Open

Fix setup-uv cache dependency config in lint and test workflows#53
smortezah wants to merge 5 commits intotoon-format:mainfrom
smortezah:fix/uv-cache

Conversation

@smortezah
Copy link
Contributor

Summary

Configure astral-sh/setup-uv cache invalidation explicitly in the lint workflow.

Changes

  • keep enable-cache: true
  • add cache-dependency-glob: pyproject.toml

Why

The workflow enables uv caching, but the repository does not appear to contain the default dependency files used by setup-uv for cache keys (uv.lock or requirements*.txt). Pointing cache invalidation at pyproject.toml avoids the warning and keeps caching enabled.

@smortezah smortezah requested a review from a team as a code owner March 16, 2026 11:21
Copilot AI review requested due to automatic review settings March 16, 2026 11:21
@smortezah smortezah changed the title Fix setup-uv cache dependency config in lint workflow Fix setup-uv cache dependency config in lint and test workflows Mar 16, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds cache-dependency-glob: pyproject.toml to setup-uv in both the lint and test workflows so the uv cache key is based on pyproject.toml instead of missing default files. Also adds a Copilot PR description template.

Changes:

  • Add cache-dependency-glob: pyproject.toml to setup-uv in lint.yml and test.yml
  • Add .copilot-pull-request-description-instructions.md PR template

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/lint.yml Add cache dependency glob for uv
.github/workflows/test.yml Add cache dependency glob for uv
.copilot-pull-request-description-instructions.md New PR description template for Copilot

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

uses: astral-sh/setup-uv@v7
with:
enable-cache: true
cache-dependency-glob: pyproject.toml
Copy link

@alesanfra alesanfra left a comment

Choose a reason for hiding this comment

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

It looks like pyproject.toml is already in the default dependency files according to the doc, am I missing something?

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.

3 participants