-
Notifications
You must be signed in to change notification settings - Fork 111
Adds Context to contain settings, https resolvers and signers. #1631
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
base: main
Are you sure you want to change the base?
Conversation
updates unit test to use this.
…write a duplicate manifest.json file.
…rts. adds unit test for from_stream and c2p2.created.
add_ingredient_from_stream always set position to zero inside read methods
added Builder::from_context() builder.with_json()
Add ContentCredential.open_stream()
…a settings instance
Reader::new and reader.with_stream added.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1631 +/- ##
==========================================
- Coverage 75.30% 75.17% -0.13%
==========================================
Files 166 169 +3
Lines 37257 37272 +15
==========================================
- Hits 28055 28018 -37
- Misses 9202 9254 +52 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
CodSpeed Performance ReportMerging #1631 will not alter performanceComparing Summary
Footnotes
|
This adds a Context struct that contains settings, https resolvers and signers.
This is a work in progress but hopefully the right direction