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(eventindexer): Event indexer metrics #13762

Merged
merged 5 commits into from
May 15, 2023
Merged

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented May 15, 2023

Also regenerates bindings for latest protocol, and subscribes to blockProposed events.

dionysuzx
dionysuzx previously approved these changes May 15, 2023
@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Merging #13762 (e40f1b2) into main (5d29c6d) will increase coverage by 2.63%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##             main   #13762      +/-   ##
==========================================
+ Coverage   45.27%   47.91%   +2.63%     
==========================================
  Files         132      131       -1     
  Lines        3538     3339     -199     
  Branches      363      310      -53     
==========================================
- Hits         1602     1600       -2     
+ Misses       1835     1638     -197     
  Partials      101      101              
Flag Coverage Δ *Carryforward flag
bridge-ui 96.16% <ø> (ø) Carriedforward from 8c7ddc4
eventindexer 81.09% <ø> (-1.00%) ⬇️
protocol 0.00% <ø> (ø) Carriedforward from 8c7ddc4
relayer 62.53% <ø> (ø) Carriedforward from 8c7ddc4
ui 100.00% <ø> (ø) Carriedforward from 8c7ddc4

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

Impacted Files Coverage Δ
packages/protocol/contracts/L1/TaikoL1.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/TaikoToken.sol 0.00% <ø> (ø)
...ckages/protocol/contracts/L1/libs/LibProposing.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/libs/LibProving.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/libs/LibUtils.sol 0.00% <ø> (ø)
...ckages/protocol/contracts/L1/libs/LibVerifying.sol 0.00% <ø> (ø)
packages/protocol/contracts/L2/TaikoL2.sol 0.00% <ø> (ø)
packages/protocol/contracts/bridge/Bridge.sol 0.00% <ø> (ø)
...ackages/protocol/contracts/bridge/BridgedERC20.sol 0.00% <ø> (ø)
packages/protocol/contracts/bridge/EtherVault.sol 0.00% <ø> (ø)
... and 32 more

... and 1 file with indirect coverage changes

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

RogerLamTd
RogerLamTd previously approved these changes May 15, 2023
@cyberhorsey cyberhorsey dismissed stale reviews from RogerLamTd and dionysuzx via 59463fc May 15, 2023 18:46
@cyberhorsey cyberhorsey enabled auto-merge May 15, 2023 19:16
@cyberhorsey cyberhorsey added this pull request to the merge queue May 15, 2023
Merged via the queue into main with commit 59ed335 May 15, 2023
@cyberhorsey cyberhorsey deleted the event_indexer_metrics branch May 15, 2023 22:03
@github-actions github-actions bot mentioned this pull request May 15, 2023
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.

3 participants