Skip to content

Conversation

@patrick-ogrady
Copy link
Contributor

No description provided.

@patrick-ogrady patrick-ogrady marked this pull request as ready for review March 10, 2025 23:59
@patrick-ogrady patrick-ogrady merged commit b7f4c96 into push-read-indexer Mar 10, 2025
4 checks passed
@patrick-ogrady patrick-ogrady deleted the indexer-trait branch March 10, 2025 23:59
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 63.33333% with 33 lines in your changes missing coverage. Please review.

Project coverage is 67.81%. Comparing base (8547a4e) to head (001edc3).
Report is 1 commits behind head on push-read-indexer.

Files with missing lines Patch % Lines
chain/src/lib.rs 66.66% 21 Missing ⚠️
chain/src/bin/validator.rs 0.00% 8 Missing ⚠️
chain/src/actors/syncer/actor.rs 77.77% 4 Missing ⚠️
@@                  Coverage Diff                  @@
##           push-read-indexer       #5      +/-   ##
=====================================================
+ Coverage              66.35%   67.81%   +1.45%     
=====================================================
  Files                     21       21              
  Lines                   3023     3082      +59     
=====================================================
+ Hits                    2006     2090      +84     
+ Misses                  1017      992      -25     
Files with missing lines Coverage Δ
chain/src/engine.rs 96.89% <100.00%> (ø)
chain/src/actors/syncer/actor.rs 88.70% <77.77%> (+6.31%) ⬆️
chain/src/bin/validator.rs 0.00% <0.00%> (ø)
chain/src/lib.rs 93.41% <66.66%> (-3.23%) ⬇️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8547a4e...001edc3. Read the comment docs.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

patrick-ogrady added a commit that referenced this pull request Mar 11, 2025
* add client import

* add indexer URL to readme

* push through indexer

* push into syncer (need blocks)

* spike on indexer

* add logs

* reuse reqwest client

* perform seed upload

* remove nullification

* add minimal listener

* inspector tweaks

* make indexer

* remove reserialization

* use finalizer to avoid confusion

* fix end condition

* [chain] Add `Indexer` trait (#5)

* pipe indexer into engine

* basic indexer test

* check indexer is working

* [inspector] Add remaining functionality (#7)

* cleanup inspector

* finish inspector

* refactor code

* update README

* add much faster

* add connection prep

* add debug

* add range

* fetch range

* update README

* update versions to 0.0.3

* make bump_versions executable
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.

2 participants