-
Notifications
You must be signed in to change notification settings - Fork 5
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
Gilt does not recognize tags added after the cache was cloned #142
Comments
0xDEC0DE
changed the title
Gilt does not recognize tree-ish values added after the cache was cloned
Gilt does not recognize tags added after the cache was cloned
Feb 3, 2024
Update
|
0xDEC0DE
pushed a commit
to 0xDEC0DE/gilt
that referenced
this issue
Feb 3, 2024
Special care and handling is needed to keep tags up-to-date with the remote; all other assets seem to be fetched/updated as needed. Fixes: Issue retr0h#142
0xDEC0DE
pushed a commit
to 0xDEC0DE/gilt
that referenced
this issue
Feb 3, 2024
Special care and handling is needed to keep tags up-to-date with the remote; all other assets seem to be fetched/updated as needed. Fixes: Issue retr0h#142
retr0h
pushed a commit
that referenced
this issue
Feb 3, 2024
Special care and handling is needed to keep tags up-to-date with the remote; all other assets seem to be fetched/updated as needed. Fixes: Issue #142
Ahh good catch, py-gilt had the update functionality to facilitate branch support. Let me know if you want me to turn around and release a bug fix release sooner than later. |
It's easy enough to work around, and our typical use case is in CI pipelines, which pop cache as part of cleanup; we're in no hurry. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Spotted when trying to vendor in a fresh change; reproduction is a bit weird:
Steps to reproduce
Setup
Verify setup
Finish setup
Test
Expected result
target
directory containsmonkey
andbanana
Actual behavior
Workaround
...will update the cache with current data, and the overlay will succeed.
The text was updated successfully, but these errors were encountered: