Skip to content

Conversation

sleepdefic1t
Copy link
Contributor

Summary

  • move Configuration argument to initial builder constructor.
  • enable passing Configuration arguments to the builder constructor.
  • cleanup builder calls in unit tests.
  • update documentation to match changes.
  • update the CHANGELOG.

Checklist

  • Documentation
  • Tests
  • Ready to be merged

- move network configuration to the initial builder constructor.
  - e.g. `builder::Transfer().build(config);` -> `builder::Transfer(config).build();`
- update unit test.
- update docs.
Enable passing Configuration constructor arguments to the builder constructor.

- rm `explicit` in `Configuration` constructors.
- rm unnecessary builder arguments from unit tests and documentation.
@ghost ghost added Complexity: Medium Less than 256 lines changed. Type: Task The issue is a request to setup third-party integrations or any general non-code related tasks. labels Feb 24, 2020
@codecov
Copy link

codecov bot commented Feb 24, 2020

Codecov Report

Merging #215 into develop will increase coverage by 0.17%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #215      +/-   ##
===========================================
+ Coverage    96.92%   97.09%   +0.17%     
===========================================
  Files           62       62              
  Lines         1170     1172       +2     
===========================================
+ Hits          1134     1138       +4     
+ Misses          36       34       -2     
Impacted Files Coverage Δ
src/transactions/builders/common.hpp 100.00% <0.00%> (+10.00%) ⬆️

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 0746433...241e3ae. Read the comment docs.

@sleepdefic1t
Copy link
Contributor Author

@faustbrian
good to go 👍

@faustbrian faustbrian merged commit 2b07c52 into develop Feb 26, 2020
@ghost ghost deleted the chore(builder)/improvements branch February 26, 2020 14:21
sleepdefic1t added a commit that referenced this pull request Mar 6, 2020
- added Delegate Resignation Builder and Tests [#207]
- improved transaction builder [#215]
- improved building as a project dependency [#206]
- make htlc claim secret a 32-byte hash [#212]
@sleepdefic1t sleepdefic1t mentioned this pull request Mar 6, 2020
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium Less than 256 lines changed. Type: Task The issue is a request to setup third-party integrations or any general non-code related tasks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants