Skip to content

Update Benchmark Repo#807

Merged
DenhamPreen merged 1 commit intomainfrom
nb/benchmark-link-update
Mar 3, 2026
Merged

Update Benchmark Repo#807
DenhamPreen merged 1 commit intomainfrom
nb/benchmark-link-update

Conversation

@nikbhintade
Copy link
Member

@nikbhintade nikbhintade commented Mar 3, 2026

Summary by CodeRabbit

  • Documentation
    • Updated benchmark documentation references to point to the Open Indexer Benchmark repository and templates.

@vercel
Copy link

vercel bot commented Mar 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
envio-docs Ready Ready Preview, Comment Mar 3, 2026 8:13am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5b3d859 and 4b654c1.

📒 Files selected for processing (1)
  • docs/HyperIndex/benchmarks.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/HyperIndex/benchmarks.md

📝 Walkthrough

Walkthrough

A documentation update in the benchmarks guide replaces a reference to the Sentio benchmark repository with the Open Indexer Benchmark repository, removing mention of sample indexer implementations. One line modified.

Changes

Cohort / File(s) Summary
Documentation Update
docs/HyperIndex/benchmarks.md
Updated benchmark repository reference from Sentio to Open Indexer Benchmark in the "Verify For Yourself" section; removed mention of sample indexer implementations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • moose-code

Poem

🐰 A reference springs forth, fresh and new,
From Sentio old to Indexer true,
The benchmark path now clearer shines,
Templates guide through cleaner lines!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update Benchmark Repo' directly corresponds to the main change: updating the benchmark repository reference in the documentation from Sentio to Open Indexer Benchmark.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch nb/benchmark-link-update

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/HyperIndex/benchmarks.md`:
- Line 43: The link in the benchmarks doc contains a malformed URL
("htthttps://github.com/enviodev/open-indexer-benchmark"); edit the text in
docs/HyperIndex/benchmarks.md to correct the URL to
"https://github.com/enviodev/open-indexer-benchmark" so the markdown link target
is valid and the anchor text remains unchanged.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 26f92c2 and 4086646.

📒 Files selected for processing (4)
  • docs/HyperIndex/benchmarks.md
  • docs/HyperRPC/hyperrpc-supported-networks.md
  • docs/HyperSync/hypersync-supported-networks.md
  • supported-networks.json
💤 Files with no reviewable changes (2)
  • supported-networks.json
  • docs/HyperRPC/hyperrpc-supported-networks.md

Copy link
Contributor

@DenhamPreen DenhamPreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the other files, it's outdated

Copy link
Contributor

@DenhamPreen DenhamPreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌

@DenhamPreen DenhamPreen merged commit e74bc21 into main Mar 3, 2026
3 checks passed
@DenhamPreen DenhamPreen deleted the nb/benchmark-link-update branch March 3, 2026 14:24
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