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

fix(relayer): only wait for confirmations when not crawling past blocks #16228

Merged
merged 1 commit into from
Mar 2, 2024

Conversation

cyberhorsey
Copy link
Contributor

crawling past blocks is slow and uses unnecessary RPC calls when it doesnt need to. this fixes it.

Copy link

codecov bot commented Mar 1, 2024

Codecov Report

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

❗ No coverage uploaded for pull request base (alpha-6@3416d9b). Click here to learn what that means.

Files Patch % Lines
packages/relayer/indexer/handle_event.go 0.00% 15 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             alpha-6   #16228   +/-   ##
==========================================
  Coverage           ?   27.03%           
==========================================
  Files              ?      102           
  Lines              ?     6096           
  Branches           ?        0           
==========================================
  Hits               ?     1648           
  Misses             ?     4268           
  Partials           ?      180           
Flag Coverage Δ *Carryforward flag
eventindexer 17.28% <ø> (?) Carriedforward from 739a268
guardian-prover-health-check 36.13% <ø> (?) Carriedforward from 739a268
relayer 37.92% <0.00%> (?)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cyberhorsey cyberhorsey merged commit ae26d8a into alpha-6 Mar 2, 2024
5 checks passed
@cyberhorsey cyberhorsey deleted the crawler_hotfix branch March 2, 2024 02:16
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