Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Sync trait to TrustHandlerConfig #440

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

dkozma
Copy link
Contributor

@dkozma dkozma commented Apr 3, 2024

Changes in this pull request

This is required for c2pa-node to upgrade to the latest version so that we can share the global config across threads.

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.

@dkozma dkozma requested review from dyro and gpeacock April 3, 2024 19:36
@codecov-commenter
Copy link

codecov-commenter commented Apr 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.26%. Comparing base (5e37c4a) to head (dead581).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #440      +/-   ##
==========================================
- Coverage   80.28%   80.26%   -0.03%     
==========================================
  Files          82       82              
  Lines       25758    25758              
==========================================
- Hits        20680    20674       -6     
- Misses       5078     5084       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@dyro dyro left a comment

Choose a reason for hiding this comment

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

@dkozma dkozma merged commit e8d87a3 into main Apr 3, 2024
22 checks passed
@dkozma dkozma deleted the add-sync-to-trust-handler-config branch April 3, 2024 20:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants