-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Remove blobstore from manifest builder #2782
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2782 +/- ##
==========================================
+ Coverage 60.24% 60.26% +0.01%
==========================================
Files 103 103
Lines 8022 8008 -14
==========================================
- Hits 4833 4826 -7
+ Misses 2546 2540 -6
+ Partials 643 642 -1
Continue to review full report at Codecov.
|
Hello from the future @glefloch. Would you mind rebasing? |
Hello @milosgajdos, sure, I will! |
16827ee
to
5bca240
Compare
@milosgajdos it should be good now :) |
Thanks, @glefloch. I need to allocate some time this week to properly review this PR. |
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.
LGTM
@glefloch can you please rebase? It does not seem to be necessary, but I'd like the workflow to rekick. Thanks! |
Sure, done! |
Are you sure? |
@glefloch do you mind rebasing 🙇♂️ |
49f500e
to
003dd5a
Compare
This PR aims to close #2150 by removing
blobstore
from schema2 manifest builder.The builder now take configuration
Descriptor
as parameter instead of a blobstore and media type.Signed-off-by: glefloch glfloch@gmail.com