-
Notifications
You must be signed in to change notification settings - Fork 86
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
Export Internal Packages + Miscellaneous Bug Fixes #93
Conversation
Pull Request Test Coverage Report for Build 3352
💛 - Coveralls |
ecddc06
to
ad0f799
Compare
|
||
// CoinStorageHelper implements the storage.CoinStorageHelper | ||
// interface. | ||
type CoinStorageHelper struct { |
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.
Should this file be in storage package if this implements storage.CoinStorageHelper
?
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.
We need to do a big refactor on everything in processor
. The whole package can likely be removed by aligning helper/handler interfaces with existing package interfaces. We would then just provide an instantiation of the fetcher
(for example) instead of a processor
that wraps the fetcher
.
Closes: #91
In this PR, all
/internal
packages are made external. A few developers have asked to use some of the storage-based packages for their own projects.Changes
/internal
to/pkg
*types.BlockIdentifier
withCoinStorage.GetCoins
check:data