Skip to content

abci: remove counter app#6684

Merged
tac0turtle merged 17 commits intomasterfrom
remove/counter
Jul 12, 2021
Merged

abci: remove counter app#6684
tac0turtle merged 17 commits intomasterfrom
remove/counter

Conversation

@tac0turtle
Copy link
Contributor

@tac0turtle tac0turtle commented Jul 9, 2021

Description

Remove counter app

ref #5790

@codecov
Copy link

codecov bot commented Jul 9, 2021

Codecov Report

Merging #6684 (4b82f64) into master (aa4854f) will increase coverage by 0.15%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #6684      +/-   ##
==========================================
+ Coverage   60.89%   61.04%   +0.15%     
==========================================
  Files         297      297              
  Lines       28228    28222       -6     
==========================================
+ Hits        17189    17228      +39     
+ Misses       9308     9268      -40     
+ Partials     1731     1726       -5     
Impacted Files Coverage Δ
cmd/tendermint/commands/run_node.go 0.00% <0.00%> (ø)
proxy/client.go 25.00% <ø> (+3.57%) ⬆️
libs/events/events.go 92.94% <0.00%> (-5.89%) ⬇️
internal/p2p/switch.go 63.63% <0.00%> (-2.28%) ⬇️
internal/mempool/v0/clist_mempool.go 81.25% <0.00%> (-1.25%) ⬇️
internal/blockchain/v0/pool.go 79.21% <0.00%> (-1.08%) ⬇️
internal/blockchain/v2/reactor.go 29.90% <0.00%> (-0.97%) ⬇️
internal/consensus/state.go 67.41% <0.00%> (-0.10%) ⬇️
internal/p2p/transport_mconn.go 82.26% <0.00%> (ø)
internal/statesync/syncer.go 78.51% <0.00%> (+0.74%) ⬆️
... and 10 more

@tac0turtle tac0turtle marked this pull request as ready for review July 9, 2021 14:58
tac0turtle and others added 4 commits July 9, 2021 15:20
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
@tac0turtle tac0turtle self-assigned this Jul 12, 2021
@tac0turtle tac0turtle added the C:abci Component: Application Blockchain Interface label Jul 12, 2021
@tac0turtle tac0turtle merged commit 363ea56 into master Jul 12, 2021
@tac0turtle tac0turtle deleted the remove/counter branch July 12, 2021 14:55
cmwaters added a commit that referenced this pull request Aug 1, 2022
* remove counter app

* remove unneeeded ci

* lint fix

* modify tx sizes

* cleanup docs

* Update abci/cmd/abci-cli/abci-cli.go

Co-authored-by: Callum Waters <cmwaters19@gmail.com>

* Update docs/app-dev/getting-started.md

Co-authored-by: Callum Waters <cmwaters19@gmail.com>

* Update docs/app-dev/getting-started.md

Co-authored-by: Callum Waters <cmwaters19@gmail.com>

* bring back comment

* migrate to kvstore and not persistent

* remove unused func

* test persistent

Co-authored-by: Callum Waters <cmwaters19@gmail.com>
cmwaters added a commit that referenced this pull request Aug 3, 2022
samricotta pushed a commit that referenced this pull request Aug 9, 2022
samricotta pushed a commit that referenced this pull request Aug 12, 2022
samricotta pushed a commit that referenced this pull request Aug 12, 2022
samricotta pushed a commit that referenced this pull request Aug 12, 2022
samricotta pushed a commit that referenced this pull request Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:abci Component: Application Blockchain Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants