-
Notifications
You must be signed in to change notification settings - Fork 160
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
Refactor Trillian client with exported methods #1454
Conversation
This allows the Trillian client to be used in other parts of the codebase besides the api package. The changes included exporting the Response fields and all but one of the struct methods. Also removed ranges since these weren't used outside the api package. Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
Context is starting work on creating a stable checkpoint, where a goroutine outside of the api package will periodically fetch the latest checkpoint. |
Codecov Report
@@ Coverage Diff @@
## main #1454 +/- ##
==========================================
- Coverage 64.35% 64.31% -0.05%
==========================================
Files 82 82
Lines 7873 7864 -9
==========================================
- Hits 5067 5058 -9
Misses 2159 2159
Partials 647 647
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Lgtm
Thanks
Moved to draft because I'm not certain I need this yet. |
Ready for review! |
This allows the Trillian client to be used in other parts of the codebase besides the api package. The changes included exporting the Response fields and all but one of the struct methods. Also removed ranges since these weren't used outside the api package.
Summary
Release Note
Documentation