Skip to content
This repository has been archived by the owner on Dec 11, 2024. It is now read-only.

Treat a source asset with a manifest store as a default parent #76

Merged
merged 7 commits into from
Sep 22, 2022

Conversation

gpeacock
Copy link
Contributor

Changes in this pull request

Treat a source asset that has a manifest as a default parent ingredient, if no other parent is specfified.

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment.

@gpeacock gpeacock changed the title Gpeacock/default_parent Treat a source asset with a manifest store as a default parent Sep 22, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@66cba79). Click here to learn what that means.
The diff coverage is 75.00%.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage        ?   72.88%           
=======================================
  Files           ?        4           
  Lines           ?      295           
  Branches        ?        0           
=======================================
  Hits            ?      215           
  Misses          ?       80           
  Partials        ?        0           
Impacted Files Coverage Δ
src/main.rs 66.12% <75.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gpeacock gpeacock merged commit 0fb8119 into main Sep 22, 2022
@gpeacock gpeacock deleted the gpeacock/default_parent branch September 22, 2022 17:13
Copy link
Contributor

@mauricefisher64 mauricefisher64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

scouten-adobe pushed a commit to contentauth/c2pa-rs that referenced this pull request Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants