-
Notifications
You must be signed in to change notification settings - Fork 11
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
Feature/archive callback #24
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #24 +/- ##
===========================================
+ Coverage 63.36% 63.49% +0.13%
===========================================
Files 235 236 +1
Lines 13343 13391 +48
Branches 1288 1291 +3
===========================================
+ Hits 8455 8503 +48
Misses 4888 4888 ☔ View full report in Codecov by Sentry. |
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.
Some minor tweaks. Please see comments.
We decided to not return a key from the archive method, and also decided not to bother propgating this change to the C API just yet |
API for registering a callback function
foo(key, location)
, to be called upon archiving to the Store.