-
Notifications
You must be signed in to change notification settings - Fork 26
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
Bec v2 #20
Conversation
…which contains crypto and key functions used by BT
…lace with String implementing Stringer
Codecov Report
@@ Coverage Diff @@
## v2 #20 +/- ##
==========================================
+ Coverage 75.89% 76.46% +0.56%
==========================================
Files 20 18 -2
Lines 809 786 -23
==========================================
- Hits 614 601 -13
+ Misses 132 120 -12
- Partials 63 65 +2
Continue to review full report at Codecov.
|
adding changeToAddress (backport from v1)
bscript/script.go
Outdated
@@ -315,3 +313,20 @@ func (s *Script) PublicKeyHash() ([]byte, error) { | |||
|
|||
return parts[0], nil | |||
} | |||
|
|||
// Equals will compare the script to b and return true if they match. | |||
func (s *Script) Equals(b *Script) bool{ |
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.
remember to add tests for these
…which contains crypto and key functions used by BT
…lace with String implementing Stringer
This adds the new go-bk dependency and remove all other dependencies go-bt had.
bk contains crypto functions as well as key related functions such as bip32 etc.
bt has now been cleaned out of all these functions making it more targeted purely to transaction building and scripts.