Revert "docs: add prebuilt seid binary install option" (#44)#45
Conversation
This reverts the net changes of PR #44 (merge commit c7fa1f4), which added the prebuilt release binary as the recommended seid install path on evm/installing-seid-cli.mdx, node/index.mdx, and node/node-operators.mdx. The prebuilt binaries are not yet attached to a published sei-chain release, so the download and sha256sum verification commands in these docs would 404 for readers. #44's own merge-timing note said to hold until the first release carrying binaries ships; it landed ahead of that. Temporary revert. To re-apply once binaries are live, revert this revert (GitHub "Revert" button on the revert PR, or git revert of this commit) — a plain re-merge of the old branch will not reintroduce the content since those commits are already in main's history. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
PR SummaryLow Risk Overview The recommended prebuilt release binary flow is removed: tabbed install UIs, On the node home page, Docker moves from a full install tab to a collapsed Alternative: Docker Image accordion; minor/major update procedures drop the prebuilt-binary branches and the “don’t mix install methods” callout. Reviewed by Cursor Bugbot for commit 71d665d. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
There was a problem hiding this comment.
Documentation-only revert of #44, removing the prebuilt-binary install path across three pages because the binaries aren't yet attached to published sei-chain releases (so the download/sha256sum commands would 404). The revert is complete and internally consistent — no stray prebuilt references remain — and the rationale is sound.
Findings: 0 blocking | 5 non-blocking | 0 posted inline
Blockers
- None at the file/PR level.
Non-blocking
- The repository's
REVIEW_GUIDELINES.mdis empty/missing on the base branch, so no repo-specific standards could be applied. - The OpenAI Codex second-opinion review (
codex-review.md) is empty — that pass produced no output. - The Cursor second-opinion review (
cursor-review.md) is empty — that pass produced no output. - Minor pre-existing inconsistency reintroduced by the revert: version placeholders differ across the restored pages (
git checkout [VERSION]and[new-version]in installing-seid-cli.mdx / node-operators.mdx vsgit checkout <version-tag>in node/index.mdx). Since this is a straight revert to prior state it's not blocking, but worth normalizing in a follow-up. - This is intentionally a temporary revert; the PR description documents the correct re-apply path (revert-the-revert) once binaries ship, so no action needed now.
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
What
Reverts #44 (merge commit
c7fa1f4), which added the prebuilt release binary as the recommendedseidinstall path. Restores three pages to their pre-#44 state:evm/installing-seid-cli.mdxnode/index.mdxnode/node-operators.mdxVerified: the reverted tree is byte-identical to the pre-merge version of each file, and no
prebuiltreferences remain.Why
The prebuilt binaries aren't attached to a published sei-chain release yet. #44's own merge-timing note said to hold until the first release carrying binaries ships — it landed ahead of that, so the download +
sha256sumverification commands would 404 for readers today.Re-applying later (keep ready)
This is a temporary revert. When the first release with binaries is out, bring the docs back by reverting this PR — the GitHub Revert button on this PR, or
git revert -m 1 <this PR's merge commit>. That cleanly restores all of #44's content.The original branch
docs/prebuilt-seid-binariesis preserved on the remote as a reference snapshot of the intended final content.🤖 Generated with Claude Code