Skip to content

[node] allow registration of custom reactors while creating node#3730

Closed
ParthDesai wants to merge 4 commits intotendermint:masterfrom
loomnetwork:parth/custom-reactor-registration
Closed

[node] allow registration of custom reactors while creating node#3730
ParthDesai wants to merge 4 commits intotendermint:masterfrom
loomnetwork:parth/custom-reactor-registration

Conversation

@ParthDesai
Copy link

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

@ParthDesai ParthDesai changed the title custom reactor registration [node] allow registration of custom reactors while creating node Jun 18, 2019
@mattkanwisher
Copy link

Loomchain uses custom reactor for gaming specific features. Would love to expose more of this portion of tendermint in the future

@codecov-io
Copy link

Codecov Report

Merging #3730 into develop will increase coverage by 0.06%.
The diff coverage is 71.42%.

@@             Coverage Diff             @@
##           develop    #3730      +/-   ##
===========================================
+ Coverage    63.87%   63.94%   +0.06%     
===========================================
  Files          241      241              
  Lines        19985    19989       +4     
===========================================
+ Hits         12766    12781      +15     
+ Misses        6171     6164       -7     
+ Partials      1048     1044       -4
Impacted Files Coverage Δ
node/node.go 63.71% <71.42%> (-0.1%) ⬇️
consensus/reactor.go 72.41% <0%> (+0.11%) ⬆️
blockchain/pool.go 80.59% <0%> (+0.32%) ⬆️
privval/signer_service_endpoint.go 89.09% <0%> (+3.63%) ⬆️
privval/signer_validator_endpoint.go 85.55% <0%> (+10%) ⬆️

Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

👍

@melekes
Copy link
Contributor

melekes commented Jul 3, 2019

Thank you for contributing this 👍 I think it's time to refactor NewNode to use functional options https://dave.cheney.net/2014/10/17/functional-options-for-friendly-apis to make it more flexible and extensible in the future. With this in mind I am going to close this PR.

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.

5 participants