Skip to content

[demo for Odran] Update README.md - #648

Closed
jmg-duarte wants to merge 2 commits into
cowprotocol:mainfrom
jmg-duarte:solana
Closed

[demo for Odran] Update README.md#648
jmg-duarte wants to merge 2 commits into
cowprotocol:mainfrom
jmg-duarte:solana

Conversation

@jmg-duarte

@jmg-duarte jmg-duarte commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Description

Changes

  • ...
  • ...

Summary by CodeRabbit

  • Documentation
    • Added a “Live on Solana!” status note to the main README.
    • Updated the CoW DAO mission statement to reference both Ethereum and Solana.

@jmg-duarte
jmg-duarte requested a review from a team as a code owner July 28, 2026 09:15
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

@jmg-duarte is attempting to deploy a commit to the cow Team on Vercel.

A member of the Team first needs to authorize it.

@jmg-duarte jmg-duarte changed the title Update README.md [demo for Odran] Update README.md Jul 28, 2026
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

README documentation now announces “Live on Solana!”, and the CoW DAO mission statement covers both Ethereum and Solana.

Changes

Solana documentation updates

Layer / File(s) Summary
Update Solana messaging
README.md, docs/README.md
Adds a Solana status line and expands the CoW DAO mission statement to include Solana.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: raq0x, cowmarketing

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is just the template placeholders and does not explain the change or list actual updates. Fill in the Description and Changes sections with a brief rationale and concrete bullet points for the README updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the main change by calling out a README update, though the demo prefix adds noise.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@jmg-duarte jmg-duarte closed this Jul 28, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/README.md`:
- Line 8: Update the introductory CoW DAO mission sentence to say “products on
Ethereum and Solana” instead of “products in Ethereum & Solana,” preserving the
rest of the wording.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4109c01b-47be-4fae-a32e-280d0db136ea

📥 Commits

Reviewing files that changed from the base of the PR and between 99f2123 and 1943b12.

📒 Files selected for processing (2)
  • README.md
  • docs/README.md

Comment thread docs/README.md
# CoW DAO

CoW DAO is on a mission to innovate the most user-protective products in Ethereum. 
CoW DAO is on a mission to innovate the most user-protective products in Ethereum & Solana

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Use “on Ethereum and Solana” here.

“Products in Ethereum & Solana” is awkward wording for blockchain deployments. Please revise this to “products on Ethereum and Solana” for clarity.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/README.md` at line 8, Update the introductory CoW DAO mission sentence
to say “products on Ethereum and Solana” instead of “products in Ethereum &
Solana,” preserving the rest of the wording.

@github-actions github-actions Bot locked and limited conversation to collaborators Jul 28, 2026
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.

1 participant