Skip to content
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

Add render option to BlockChain<T>() constructor #883

Merged
merged 7 commits into from
May 29, 2020

Conversation

earlbread
Copy link
Contributor

This adds allowRender option to BlockChain<T>.

CHANGES.md Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
@earlbread earlbread force-pushed the allow-render branch 2 times, most recently from bb75f9e to 75ba39b Compare May 28, 2020 02:42
@codecov
Copy link

codecov bot commented May 28, 2020

Codecov Report

Merging #883 into 9c-beta will decrease coverage by 0.26%.
The diff coverage is 90.32%.

@@             Coverage Diff             @@
##           9c-beta     #883      +/-   ##
===========================================
- Coverage    87.51%   87.25%   -0.27%     
===========================================
  Files          251      247       -4     
  Lines        22826    22184     -642     
===========================================
- Hits         19977    19357     -620     
+ Misses        1517     1500      -17     
+ Partials      1332     1327       -5     
Impacted Files Coverage Δ
Libplanet/Net/Swarm.cs 85.20% <0.00%> (-0.28%) ⬇️
Libplanet.Tests/Blockchain/BlockChainTest.cs 98.53% <100.00%> (+0.03%) ⬆️
Libplanet.Tests/Store/StoreTest.cs 98.14% <100.00%> (+<0.01%) ⬆️
Libplanet/Blockchain/BlockChain.cs 90.64% <100.00%> (+0.10%) ⬆️
Libplanet/Blocks/BlockDigest.cs 76.31% <0.00%> (-10.53%) ⬇️
Libplanet/Net/Protocols/KademliaProtocol.cs 64.63% <0.00%> (-0.39%) ⬇️
Libplanet.RocksDBStore.Tests/RocksDBStoreTest.cs
...ibplanet.RocksDBStore.Tests/RocksDBStoreFixture.cs
Libplanet.RocksDBStore/RocksDBStore.cs
Libplanet.RocksDBStore/RocksDBStoreBitConverter.cs

@earlbread earlbread changed the title Allow render Add allowRender BlockChain<T> May 28, 2020
Libplanet/Blockchain/BlockChain.cs Outdated Show resolved Hide resolved
Libplanet/Blockchain/BlockChain.cs Outdated Show resolved Hide resolved
CHANGES.md Outdated Show resolved Hide resolved
@earlbread earlbread requested a review from dahlia May 28, 2020 05:13
riemannulus
riemannulus previously approved these changes May 28, 2020
moreal
moreal previously approved these changes May 28, 2020
Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

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

Sorry I missed things again in the previous review. I believe this round is the last.

Libplanet/Blockchain/BlockChain.cs Outdated Show resolved Hide resolved
@dahlia dahlia changed the title Add allowRender BlockChain<T> Add render option to BlockChain<T>() constructor May 28, 2020
@earlbread earlbread dismissed stale reviews from moreal and riemannulus via de13230 May 28, 2020 05:25
@earlbread earlbread requested a review from dahlia May 28, 2020 05:25
dahlia
dahlia previously approved these changes May 28, 2020
moreal
moreal previously approved these changes May 28, 2020
riemannulus
riemannulus previously approved these changes May 28, 2020
longfin
longfin previously approved these changes May 29, 2020
@earlbread earlbread merged commit 377e454 into planetarium:9c-beta May 29, 2020
@earlbread earlbread deleted the allow-render branch May 29, 2020 12:57
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 23, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 23, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 23, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 24, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 24, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 24, 2020
Because without the render option, the same behaviour can be achieved by
using non renderers.

See also planetarium#883
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 27, 2020
This implements the equivalent optimization made by the following issue:

planetarium#883

Fixes planetarium#967
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 28, 2020
This implements the equivalent optimization made by the following issue:

planetarium#883

Fixes planetarium#967
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 28, 2020
This implements the equivalent optimization made by the following issue:

planetarium#883

Fixes planetarium#967
dahlia added a commit to dahlia/libplanet that referenced this pull request Aug 28, 2020
This implements the equivalent optimization made by the following issue:

planetarium#883

Fixes planetarium#967
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