Skip to content

Add binding signature only if needed#221

Merged
str4d merged 1 commit into
zcash:masterfrom
adityapk00:binding_sig
Apr 16, 2020
Merged

Add binding signature only if needed#221
str4d merged 1 commit into
zcash:masterfrom
adityapk00:binding_sig

Conversation

@adityapk00

@adityapk00 adityapk00 commented Apr 3, 2020

Copy link
Copy Markdown
Contributor

Fixes #220 .

@codecov

codecov Bot commented Apr 3, 2020

Copy link
Copy Markdown

Codecov Report

Merging #221 into master will increase coverage by 0.21%.
The diff coverage is 97.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
+ Coverage   51.08%   51.30%   +0.21%     
==========================================
  Files          99       99              
  Lines       13318    13350      +32     
==========================================
+ Hits         6804     6849      +45     
+ Misses       6514     6501      -13     
Impacted Files Coverage Δ
zcash_primitives/src/transaction/builder.rs 82.73% <97.22%> (+6.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97c21e0...3ccadf3. Read the comment docs.

daira
daira previously requested changes Apr 4, 2020

@daira daira left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Needs a test that would have failed without this change. Otherwise utACK.

@adityapk00

adityapk00 commented Apr 4, 2020

Copy link
Copy Markdown
Contributor Author

Added 2 tests.

@str4d str4d left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

utACK

@str4d str4d dismissed daira’s stale review April 16, 2020 01:46

Comment addressed.

@str4d str4d merged commit b82dac1 into zcash:master Apr 16, 2020
greg0x pushed a commit to valargroup/librustzcash that referenced this pull request Mar 12, 2026
Add server heartbeat system for helper liveness tracking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tx builder adds Binding Signature even for t->t transactions

3 participants