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

feat(protocol): eventindexer/relayer/status-page/bridge-ui caught up to major protocol upgrade #13643

Merged
merged 5 commits into from
Apr 24, 2023

Conversation

cyberhorsey
Copy link
Contributor

No description provided.

@cyberhorsey cyberhorsey changed the base branch from main to major_protocol_upgrade_alpha3 April 24, 2023 15:50
@cyberhorsey cyberhorsey requested review from dantaik, dionysuzx, davidtaikocha and jscriptcoder and removed request for dantaik and dionysuzx April 24, 2023 15:50
@codecov
Copy link

codecov bot commented Apr 24, 2023

Codecov Report

Merging #13643 (a9a702d) into major_protocol_upgrade_alpha3 (642858f) will increase coverage by 0.05%.
The diff coverage is 71.42%.

❗ Current head a9a702d differs from pull request most recent head fffec92. Consider uploading reports for the commit fffec92 to get more accurate results

@@                        Coverage Diff                        @@
##           major_protocol_upgrade_alpha3   #13643      +/-   ##
=================================================================
+ Coverage                          44.80%   44.85%   +0.05%     
=================================================================
  Files                                132      132              
  Lines                               3529     3529              
  Branches                             363      363              
=================================================================
+ Hits                                1581     1583       +2     
+ Misses                              1847     1845       -2     
  Partials                             101      101              
Flag Coverage Δ *Carryforward flag
bridge-ui 95.80% <100.00%> (ø)
eventindexer 82.08% <ø> (+0.99%) ⬆️
protocol 0.00% <ø> (ø) Carriedforward from 1e83ec3
relayer 62.55% <60.00%> (ø)
ui 100.00% <ø> (ø) Carriedforward from 1e83ec3

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
...ackages/bridge-ui/src/constants/abi/IXchainSync.ts 100.00% <ø> (ø)
packages/relayer/message/wait_header_synced.go 65.11% <33.33%> (ø)
packages/bridge-ui/src/proof/ProofService.ts 100.00% <100.00%> (ø)
packages/relayer/indexer/service.go 76.04% <100.00%> (ø)
packages/relayer/message/process_message.go 46.58% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Co-authored-by: David <david@taiko.xyz>
@cyberhorsey cyberhorsey merged commit ee8e875 into major_protocol_upgrade_alpha3 Apr 24, 2023
@cyberhorsey cyberhorsey deleted the major_protocol_services branch April 24, 2023 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📝 Todo
Development

Successfully merging this pull request may close these issues.

3 participants