Skip to content

Comments

feat: uv_version to pin uv#194

Merged
cpanato merged 1 commit intohelm:mainfrom
jmelahman:jamison/uv_version
Feb 18, 2026
Merged

feat: uv_version to pin uv#194
cpanato merged 1 commit intohelm:mainfrom
jmelahman:jamison/uv_version

Conversation

@jmelahman
Copy link
Contributor

@jmelahman jmelahman commented Jan 6, 2026

Passes along the version value to setup-uv.

It can be considered best practice to pin this version a la supply-chain concerns, but more practically, this action hits the Github API if this is unset and we're getting rate-limited,

Run astral-sh/setup-uv@eb1897b8dc4b5d5bfe39a428a8f2304605e0983c
(node:1557) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
Trying to find version for uv in: /home/runner/_work/onyx/onyx/uv.toml
Could not find file: /home/runner/_work/onyx/onyx/uv.toml
Trying to find version for uv in: /home/runner/_work/onyx/onyx/pyproject.toml
Could not determine uv version from uv.toml or pyproject.toml. Falling back to latest.
Getting latest version from GitHub API...
Error: Github API request failed while getting latest release. Check the GitHub status page for outages. Try again later.
Error: API rate limit exceeded for installation. If you reach out to GitHub Support for help, please include the request ID DFFE:3AC19:A0F67B:2C82286:695D8D30 and timestamp 2026-01-06 22:31:12 UTC. For more on scraping GitHub and how it may affect your rights, please review our Terms of Service (https://docs.github.com/en/site-policy/github-terms/github-terms-of-service) - https://docs.github.com/rest/overview/rate-limits-for-the-rest-api

Closes #195

Signed-off-by: Jamison Lahman <jamison@lahman.dev>
@jmelahman
Copy link
Contributor Author

cc: @cpanato

Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm

thanks

@cpanato cpanato closed this Feb 18, 2026
@cpanato cpanato reopened this Feb 18, 2026
@cpanato cpanato merged commit b5eebdd into helm:main Feb 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow configuring the uv version

2 participants