Skip to content

Change banner to refer to Skills instead of MCP#795

Merged
ericglau merged 1 commit into
OpenZeppelin:masterfrom
ericglau:renamebannerskills
Apr 14, 2026
Merged

Change banner to refer to Skills instead of MCP#795
ericglau merged 1 commit into
OpenZeppelin:masterfrom
ericglau:renamebannerskills

Conversation

@ericglau
Copy link
Copy Markdown
Member

No description provided.

@ericglau ericglau requested a review from a team as a code owner April 14, 2026 17:36
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 63d81541-1f36-4a4a-8016-3029c6910963

📥 Commits

Reviewing files that changed from the base of the PR and between 41494aa and 256d448.

📒 Files selected for processing (7)
  • packages/ui/public/cairo.html
  • packages/ui/public/confidential.html
  • packages/ui/public/index.html
  • packages/ui/public/polkadot.html
  • packages/ui/public/stellar.html
  • packages/ui/public/stylus.html
  • packages/ui/public/uniswap-hooks.html

Walkthrough

Seven HTML files in the public directory have their promotional banner links updated to point from the MCP site (https://mcp.openzeppelin.com/) to the GitHub openzeppelin-skills repository (https://github.com/OpenZeppelin/openzeppelin-skills/), with display text changing from "Contracts MCP" to "Contracts Skills".

Changes

Cohort / File(s) Summary
Banner Link Updates
packages/ui/public/cairo.html, packages/ui/public/confidential.html, packages/ui/public/index.html, packages/ui/public/polkadot.html, packages/ui/public/stellar.html, packages/ui/public/stylus.html, packages/ui/public/uniswap-hooks.html
Updated promotional banner link href from https://mcp.openzeppelin.com/ to https://github.com/OpenZeppelin/openzeppelin-skills/ and display text from "Contracts MCP" to "Contracts Skills" across all public HTML pages.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Rename UI Builder label #673: Modifies the promotional banner copy in packages/ui/public/index.html, updating the same UI banner element with related content changes.

Suggested reviewers

  • CoveMB
  • pasevin
  • son-oz
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, but the title and changeset are clear enough to understand the purpose of updating banner references. Consider adding a brief description explaining why the banner references are being updated from MCP to Skills for better context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: updating banner links and labels across multiple HTML files from 'MCP' to 'Skills'.
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

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

Copy link
Copy Markdown

@son-oz son-oz left a comment

Choose a reason for hiding this comment

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

LGTM

@ericglau ericglau merged commit 22704ea into OpenZeppelin:master Apr 14, 2026
22 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 14, 2026
@ericglau ericglau deleted the renamebannerskills branch April 14, 2026 19:15
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.

3 participants