-
Notifications
You must be signed in to change notification settings - Fork 56
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
(MINOR) IngredientOptions
allow override of hash and thumbnail generation; image library is now a default feature
#79
Conversation
Add thumbnail option
Codecov Report
@@ Coverage Diff @@
## main #79 +/- ##
==========================================
- Coverage 74.50% 74.42% -0.09%
==========================================
Files 66 66
Lines 13471 13565 +94
==========================================
+ Hits 10037 10096 +59
- Misses 3434 3469 +35
Continue to review full report at Codecov.
|
This is required for allowing PS to not generate thumbnails via the cai-helper |
IngredientOptions
allow override of hash and thumbnail generation; image library is now a default feature
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.
👍
…timize * 'main' of https://github.com/contentauth/c2pa-rs: (MINOR) `IngredientOptions` allow override of hash and thumbnail generation; image library is now a default feature (#79) Prepare 0.9.1 release Fix publish workflow (#82) Prepare 0.9.0 release (MINOR) Introduce a new `SigningAlg` enum (#76) # Conflicts: # sdk/src/ingredient.rs
Changes in this pull request
IngredientOptions is now a trait with three methods to override title, hash and thumbnail generation
The blake3 hash is no longer built in, if a hash is desired it can be added via ingredientOptions.
add_thumbnails feature now controls if the image crate is included in a build, by default it is enabled
updated to latest image library
added additional file type/mime mappings for ingredients
Checklist
TO DO
items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.