Skip to content

Update GitHub actions, use setup-uv action #31

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
TheJulianJES merged 14 commits into from
Dec 28, 2025
Merged

Update GitHub actions, use setup-uv action #31

TheJulianJES merged 14 commits into from
Dec 28, 2025

Conversation

@TheJulianJES
Copy link
Contributor

@TheJulianJES TheJulianJES commented Dec 9, 2025 •
edited
Loading

This updates the following GitHub actions:

  • update setup-python action from v5 to v6
  • update checkout action from v4 to v6
  • update cache action from to v4 v5
  • update upload-artifact action from v4 to v6
  • update download-artifact action from v4 to v7

This also changes the CI workflow to use the setup-uv action, instead of running the install script.
Note: enable-cache is false as we cache the entire venv anyway, so I don't think we need to have UV cache something as well.
TheJulianJES
TheJulianJES commented Dec 10, 2025
View reviewed changes
@TheJulianJES
Copy link
Contributor Author

TheJulianJES commented Dec 28, 2025 •
edited
Loading

Seems to work fine, though some repos override CACHE_VERSION for some reason...
We probably want to remove that, though we can also remove cache via the UI as well.
@TheJulianJES TheJulianJES marked this pull request as ready for review December 28, 2025 07:30
@TheJulianJES TheJulianJES merged commit 07b921c into zigpy:main Dec 28, 2025
@TheJulianJES
Copy link
Contributor Author

TheJulianJES commented Dec 28, 2025 •
edited
Loading

Hmm. Looks like the new download-artifact works fine with multiple artifacts, but if there's only one artifact, it's downloaded to a different location now, so our coverage is broken. Related issue:

Will take a look...

EDIT: Fixed (or worked around) with #32
@TheJulianJES TheJulianJES mentioned this pull request Dec 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@TheJulianJES