chore(pages): remove MacPorts install channel - #1334
Merged
Merged
Conversation
Drop the MacPorts option from the landing page install switcher and the documentation site, since it is no longer a recommended way to install ocr. - Remove the macports entry and icon import from INSTALL_CHANNELS - Delete the macports.svg icon and the hero.installMacPorts label from all five locale tables - Remove the MacPorts section from every installation.md and adjust the install-method count from six to five - Switch the overflow-channel test to the Windows channel
Contributor
|
✅ OpenCodeReview: Review complete: 0 finding(s) across 6 selected item(s). |
15 tasks
wu21-web
approved these changes
Sep 16, 2026
wu21-web
left a comment
wu21-web
left a comment
Contributor
There was a problem hiding this comment.
Thanks. I told this to the port maintainer and oytech will take over the port.
stay-foolish-forever
previously approved these changes
Sep 16, 2026
lizhengfeng101
force-pushed
the
remove-macports-install
branch
from
September 17, 2026 02:05
1c0bafb to
afcb17d
Compare
wu21-web
approved these changes
Sep 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Remove the MacPorts installation method from the marketing site (
pages/) and the documentation site.HeroSection.tsx): drop themacportsentry and its icon import fromINSTALL_CHANNELS.pages/src/assets/icons/macports.svg.hero.installMacPortslabel from all five locale tables (en / zh / ja / ko / ru).## MacPortssection from everyinstallation.mdand adjust the install-method count from six to five.HeroSection.test.tsxfrom MacPorts to the Windows channel.Why
MacPorts is no longer a recommended way to install
ocr, so it should not be surfaced as an install option on the site or in the docs.Verification
HeroSectioncomponent tests pass (6/6).tsc --noEmitand ESLint report no new errors.port installreferences underpages/.AI/LLM disclosure
Model used: Claude Opus 4.8. The diff was additionally reviewed with
ocr review(open-code-review), which reported no findings.