Skip to content

Conversation

@tmathern
Copy link
Collaborator

@tmathern tmathern commented Jan 7, 2026

Changes in this pull request

Add a factory method to the Reader class:

  • if there is JUMBF (c2pa/manifest) data to read, from_asset will return a Reader instance
  • if there is no JUMBF (c2pa/manifest) data to read, returns None instead of throwing

Builds on top of #213 which improves error handling (and enables to write shorter code here).

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.

@tmathern tmathern requested a review from scouten-adobe January 7, 2026 19:56
@tmathern tmathern self-assigned this Jan 7, 2026
Base automatically changed from mathern/exceptions-refactor to vNext January 7, 2026 20:40
@tmathern tmathern merged commit a01dd37 into vNext Jan 7, 2026
20 checks passed
@tmathern tmathern deleted the mathern/try-read branch January 7, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants