Skip to content

don't try to build or restore a cache for the tox_matrix.py script#375

Merged
Cadair merged 2 commits intoOpenAstronomy:mainfrom
zacharyburnett:fix/uv_cache_warning
Mar 10, 2026
Merged

don't try to build or restore a cache for the tox_matrix.py script#375
Cadair merged 2 commits intoOpenAstronomy:mainfrom
zacharyburnett:fix/uv_cache_warning

Conversation

@zacharyburnett
Copy link
Contributor

@zacharyburnett zacharyburnett commented Mar 9, 2026

turn enable-cache: false in the setup-uv step, to not attempt cache that environment at all

@zacharyburnett zacharyburnett changed the title build uv cache against hash of script to fix annotation warnings about cache-dependency-glob don't try to build or restore a cache for the tox_matrix.py script Mar 9, 2026
@neutrinoceros
Copy link
Contributor

what's the motivation here exactly ?
Relatedly, if we ever use setup-uv in publishing workflows, I would advise to disable cache from the get go to avoid cache poisoning attacks.

@Cadair
Copy link
Member

Cadair commented Mar 10, 2026

This is to remove warnings from the build summary. See this: https://github.com/OpenAstronomy/github-actions-workflows/actions/runs/22856780197

@Cadair Cadair force-pushed the fix/uv_cache_warning branch from 8aa4f36 to 58e80e4 Compare March 10, 2026 10:17
@Cadair Cadair merged commit 1d94feb into OpenAstronomy:main Mar 10, 2026
73 checks passed
@zacharyburnett zacharyburnett deleted the fix/uv_cache_warning branch March 10, 2026 13:22
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