Skip to content

Conversation

@glokos
Copy link

@glokos glokos commented Nov 9, 2025

Summary by CodeRabbit

  • Documentation
    • Updated MEV Blocker concept doc: removed an outdated transaction volume claim and clarified that MEV Blocker is the only RPC offering a permissionless Order Flow Auction where both searchers and builders operate under public rules.

Removed redundant sentence about MEV Blocker processing transactions.
@glokos glokos requested a review from a team as a code owner November 9, 2025 19:18
@vercel
Copy link

vercel bot commented Nov 9, 2025

@glokos is attempting to deploy a commit to the cow Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 9, 2025

Walkthrough

Documentation sentence in the gas rebates conceptual guide was replaced: the previous claim about processing ~10% of daily Ethereum transactions was removed and replaced with a statement that MEV Blocker operates the only permissionless Order Flow Auction binding searchers and builders to public rules.

Changes

Cohort / File(s) Change Summary
Documentation Update
docs/mevblocker/concepts/gas_rebates.md
Removed a transaction-volume claim and replaced it with a corrected description stating MEV Blocker is the only RPC offering a permissionless Order Flow Auction where both searchers and builders are bound to public rules.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single-file documentation text replacement; no code, tests, or behavioral changes.
  • Review focus: ensure phrasing accuracy and no unintended edits.

Poem

🐰 A line trimmed gentle, snug and bright,
Replaced with rules that stand in light,
Builders, searchers bound to play,
In public halls they find their way,
A rabbit hops — the docs delight! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely missing; no description was provided by the author despite a template being available in the repository. Provide a pull request description following the repository template, including a description of changes and the rationale for the content refinement.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'doc: refine gas_rebates.md content for clarity' accurately describes the main change in the pull request—refining documentation content in the gas_rebates.md file for clarity.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ed4fd31 and 667f837.

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

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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3ebdab0 and ed4fd31.

📒 Files selected for processing (1)
  • docs/mevblocker/concepts/gas_rebates.md (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-06-30T14:12:14.869Z
Learnt from: pretf00d
Repo: cowprotocol/docs PR: 512
File: docs/mevblocker/concepts/gas_rebates.md:16-16
Timestamp: 2025-06-30T14:12:14.869Z
Learning: When referring to MEV Blocker or other products/services in documentation, use "that" instead of "who" as the relative pronoun (e.g., "MEV Blocker is the only RPC that has..." not "MEV Blocker is the only RPC who has...") because products are not people.

Applied to files:

  • docs/mevblocker/concepts/gas_rebates.md

Copy link
Contributor

@pretf00d pretf00d left a comment

Choose a reason for hiding this comment

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

One small lower case issue to address.

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@glokos glokos requested a review from pretf00d November 11, 2025 12:57
@pretf00d pretf00d closed this Dec 1, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants