-
Notifications
You must be signed in to change notification settings - Fork 565
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
Drop tuf client dependency on GCS client library #1967
Conversation
Signed-off-by: Jason Hall <jason@chainguard.dev>
Codecov Report
@@ Coverage Diff @@
## main #1967 +/- ##
==========================================
- Coverage 34.80% 34.66% -0.15%
==========================================
Files 153 152 -1
Lines 10119 10089 -30
==========================================
- Hits 3522 3497 -25
+ Misses 6226 6222 -4
+ Partials 371 370 -1
Continue to review full report at Codecov.
|
Signed-off-by: Jason Hall <jason@chainguard.dev>
cc @asraa |
Signed-off-by: Jason Hall <jason@chainguard.dev>
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.
Can we just use an HTTP remote store?
If it's a little hard to understand, I can take a closer look too
Oops, did not mean to approve, just meant to comment |
Signed-off-by: Jason Hall <jason@chainguard.dev>
That seems like a good idea. I believe the previous behavior was just to interpret a mirror name like |
Exactly!! Let's do it! The less code the less... bugs? (I hope!) Thank you :D |
This change cuts 2MB off the size of
|
NICE |
Signed-off-by: Jason Hall <jason@chainguard.dev>
Signed-off-by: Jason Hall <jason@chainguard.dev>
Nooo, one last docgen update |
Signed-off-by: Jason Hall <jason@chainguard.dev>
Signed-off-by: Jason Hall jason@chainguard.dev
cc @haydentherapper @asraa