Feat/rollback #38

sh-cha · 2024-03-29T07:18:12Z

No description provided.

…ge plan preset

…s changed

x/opchild/keeper/executor_change.go

…d of DA light node

sh-cha · 2024-04-01T08:36:21Z

WIP: need to write readme & env sample about batch submitter

…proposer, challenger, batchinfo

bots/src/celestia/utils.ts

bots/src/lib/tx.ts

* register denom metadata * fix tests * register base denom and fix claim response * add l1 denom in resurrector * add executor monitor start height * bug fix: add fee param as undefined to adjust sendtx change * delete output proposal only if it is not finalized * add tests * add tests and minor type fixes * validate bridge config before update * enforce to set batch info * fix error message * cleanup --------- Co-authored-by: JSHan94 <[email protected]> Co-authored-by: ALPAC-4 <[email protected]> Co-authored-by: hoon <[email protected]>

beer-1

LGTM

codecov · 2024-04-03T10:01:53Z

Codecov Report

Attention: Patch coverage is 31.98381% with 168 lines in your changes are missing coverage. Please review.

Project coverage is 55.69%. Comparing base (8e6f980) to head (8e8060c).
Report is 8 commits behind head on main.

❗ Current head 8e8060c differs from pull request most recent head 096484b. Consider uploading reports for the commit 096484b to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   58.10%   55.69%   -2.41%     
==========================================
  Files          23       30       +7     
  Lines        1475     2361     +886     
==========================================
+ Hits          857     1315     +458     
- Misses        437      788     +351     
- Partials      181      258      +77

Files	Coverage Δ
x/opchild/keeper/keeper.go	`81.35% <100.00%> (+0.99%)`	⬆️
x/ophost/client/cli/tx.go	`62.18% <100.00%> (ø)`
x/opchild/keeper/val_state_change.go	`52.38% <40.00%> (-1.95%)`	⬇️
x/ophost/keeper/bridge.go	`50.98% <0.00%> (-3.19%)`	⬇️
x/ophost/keeper/querier.go	`76.28% <66.66%> (-0.99%)`	⬇️
x/opchild/keeper/historical_info.go	`47.61% <16.66%> (-1.22%)`	⬇️
x/ophost/keeper/output.go	`57.74% <59.09%> (+0.60%)`	⬆️
x/ophost/keeper/msg_server.go	`59.39% <63.79%> (+0.56%)`	⬆️
x/opchild/keeper/executor_change.go	`26.31% <26.31%> (ø)`
x/opchild/client/cli/genesis.go	`0.00% <0.00%> (ø)`

... and 4 files with indirect coverage changes

sh-cha added 10 commits March 21, 2024 19:04

add opinit pulsar

f42c8ad

bug fix: add fee param as undefined to adjust sendtx change

1cd3d7c

proto api update

1037fc0

delete output proposal only if it is not finalized

8097c25

add ophost autocli query

90c5fc0

change executor's validator at end blocker according to executor chan…

4253a61

…ge plan preset

batch executor send last block's commit appending end of blocks bulk

eab8ac0

add last finalized output querier to ophost

7269859

rpc raw commit param height

9ff00eb

change old executor's conspower as -1 to let comet know if executor i…

c9784a2

…s changed

sh-cha requested a review from beer-1 March 29, 2024 07:18

sh-cha requested a review from a team as a code owner March 29, 2024 07:18

beer-1 reviewed Apr 1, 2024

View reviewed changes

x/opchild/keeper/executor_change.go Outdated Show resolved Hide resolved

batch submitter to celestia send tx to celestia consensus node instea…

46c9294

…d of DA light node

sh-cha added 4 commits April 1, 2024 18:31

delete all validators & set next executor's validator

fb039fe

add batch info to bridge config

b27b7d7

emit and return finalized output index, l2 blocknumber when updating …

74a1a0f

…proposer, challenger, batchinfo

emit finalized output event

8b8597a