Skip to content

Correct the doc comment and add a test for Builder::value_balance#359

Merged
nuttycom merged 2 commits intozcash:mainfrom
daira:add-value-balance-to-builder-docfix
Oct 10, 2022
Merged

Correct the doc comment and add a test for Builder::value_balance#359
nuttycom merged 2 commits intozcash:mainfrom
daira:add-value-balance-to-builder-docfix

Conversation

@daira
Copy link
Copy Markdown
Contributor

@daira daira commented Sep 19, 2022

Follow-up to #352.

@daira daira added the documentation Improvements or additions to documentation label Sep 19, 2022
@daira daira requested a review from str4d September 19, 2022 12:28
@daira daira changed the title Correct the doc comment for Builder::value_balance added in #352 Correct documentation and add test for Builder::value_balance Sep 19, 2022
@daira daira marked this pull request as draft September 19, 2022 12:31
@daira daira changed the title Correct documentation and add test for Builder::value_balance Correct documentation for Builder::value_balance Sep 19, 2022
@daira daira changed the title Correct documentation for Builder::value_balance Correct the doc comment and add a test for Builder::value_balance Sep 19, 2022
@daira daira marked this pull request as ready for review September 19, 2022 12:53
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Base: 88.61% // Head: 88.61% // No change to project coverage 👍

Coverage data is based on head (899a9eb) compared to base (f206b3f).
Patch has no changes to coverable lines.

❗ Current head 899a9eb differs from pull request most recent head e550278. Consider uploading reports for the commit e550278 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #359   +/-   ##
=======================================
  Coverage   88.61%   88.61%           
=======================================
  Files          38       38           
  Lines        3875     3875           
=======================================
  Hits         3434     3434           
  Misses        441      441           
Impacted Files Coverage Δ
src/builder.rs 72.50% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@daira daira requested a review from nuttycom September 19, 2022 12:59
… in zcash#352.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
Signed-off-by: Daira Hopwood <daira@jacaranda.org>
@daira daira force-pushed the add-value-balance-to-builder-docfix branch from e550278 to 6458e27 Compare September 19, 2022 13:06
@daira daira added this to the Release 5.3.0 milestone Sep 19, 2022
Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

utACK

@nuttycom nuttycom merged commit 33bce9b into zcash:main Oct 10, 2022
@daira daira deleted the add-value-balance-to-builder-docfix branch November 22, 2022 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants