-
Notifications
You must be signed in to change notification settings - Fork 77
Resolve latest version instead of downloading latest release #178
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.
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
Conversation
|
Thank you for your contribution! Currently on my phone so a dedicated review will follow. Can you already provide more detail on how the new output can be used? |
eifinger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice addition. I like it alot!
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
f219245 to
d59153e
Compare
Co-authored-by: Kevin Stillhammer <kevin.stillhammer@gmail.com>
eifinger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really awesome. Thank you! 🎆
Summary
Hi, this is my first contribution.
I addressed issue #171 by adjusting the uv
version: latestbehavior. I also addeduv-cache-hitoutput and included CI tests (see attached image).I look forward to your feedback.
Fig.1: success CI
Changes
download-cachedownload-version process.