Skip to content

Update settle call block deadline#637

Open
harisang wants to merge 1 commit into
mainfrom
update_settle_call_block_deadline
Open

Update settle call block deadline#637
harisang wants to merge 1 commit into
mainfrom
update_settle_call_block_deadline

Conversation

@harisang

@harisang harisang commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

This PR updates the settle call block deadline info in the solvers documentation

Summary by CodeRabbit

  • Documentation
    • Updated auction competition deadline specifications across multiple supported blockchain networks including Arbitrum, Base, Avalanche, Polygon, BNB, Linea, Plasma, and Ink, with refined block count configurations for each network.

@harisang harisang requested a review from a team as a code owner June 21, 2026 00:12
@vercel

vercel Bot commented Jun 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Jun 21, 2026 12:12am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1cac1fdf-f393-4f7a-a453-8579277816d1

📥 Commits

Reviewing files that changed from the base of the PR and between 978e255 and e6f459a.

📒 Files selected for processing (1)
  • docs/cow-protocol/reference/core/auctions/competition_rules.md

📝 Walkthrough

Walkthrough

Updates the "deadline for solutions" block-count values in competition_rules.md. The per-network list now covers additional chains (Arbitrum, Base, Avalanche, Polygon, BNB, Linea, Plasma, Ink) with revised numeric block values, replacing the previous chain set.

Changes

Competition Rules Deadline Update

Layer / File(s) Summary
Per-network solution deadline block counts
docs/cow-protocol/reference/core/auctions/competition_rules.md
Replaces the prior deadline block-count mapping with an updated list that adds networks (including Ink) and changes the numeric block values for listed chains.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through chains galore,
Counting blocks from every shore,
Ink and Base and Polygon too,
Deadlines set with values new.
The docs are tidy, fresh, and bright —
Every chain timed just right! 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal but provides context. However, it lacks the structured format specified in the template with separate 'Description' and 'Changes' sections with detailed change items. Expand the description to follow the template structure: add a 'Description' section with detailed context on why the deadline updates are needed, and a 'Changes' section with bullet points listing specific chain deadline updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: updating settle call block deadline information in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update_settle_call_block_deadline

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.

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.

1 participant