Repository navigation
Move the website to getlantern/soar-site - #11
Merged
Merged
Conversation
The site now lives in its own repository, with its history, and Cloudflare Pages deploys soar.lantern.io from there. Removing it here keeps about 276 KB of HTML and fonts out of the Go module that every importer downloads. The README links the site's new repo and the Rust implementation. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QQFpfCXs3JqrfyDuiGabF5
Deploying soar with
|
| Latest commit: |
9e94848
|
| Status: | 🚫 Build failed. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly describes the main change: moving the website to the getlantern/soar-site repository and removing the site files from this repository. |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
✨ Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
There was a problem hiding this comment.
🟢 Approval recommended
The migrated assets match the destination repository, references are updated, and no remaining build configuration depends on site/.
0 open findings
What changed in this PR
Moves the static website to getlantern/soar-site, reducing the Go module footprint while retaining project links.
Changes:
- Removes the complete
site/tree. - Links the new website repository and Rust implementation from the README.
| File | Description |
|---|---|
README.md |
Adds website-source and Rust implementation links. |
site/README.md |
Removes obsolete local deployment documentation. |
site/_headers |
Removes migrated Cloudflare headers. |
site/fonts.css |
Removes migrated font definitions. |
site/fonts/OFL-literata.txt |
Removes migrated Literata license. |
site/fonts/OFL-martianmono.txt |
Removes migrated Martian Mono license. |
site/fonts/OFL-sairacondensed.txt |
Removes migrated Saira license. |
site/fonts/literata-italic-0.woff2 |
Removes migrated font asset. |
site/fonts/literata-normal-1.woff2 |
Removes migrated font asset. |
site/fonts/martian-mono-normal-2.woff2 |
Removes migrated font asset. |
site/fonts/saira-condensed-normal-3.woff2 |
Removes migrated font asset. |
site/fonts/saira-condensed-normal-4.woff2 |
Removes migrated font asset. |
site/fonts/saira-condensed-normal-5.woff2 |
Removes migrated font asset. |
site/fonts/saira-condensed-normal-6.woff2 |
Removes migrated font asset. |
site/index.html |
Removes the migrated static website. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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
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.
Why
The website is moving to its own repository, getlantern/soar-site, with its history (via
git subtree split). Two reasons:github.com/getlantern/soarwere fetching about 276 KB of HTML and fonts, more than a third of the module.What
site/.Deployment
soar.lantern.ionow belongs to a new Cloudflare Pages project,soar-site, which buildsgetlantern/soar-site. The oldsoarproject here has no custom domain any more. It's left in place but does nothing useful, and can be deleted once the new one is serving.Pre-PR review (local Codex gate)
CODEX GATE: verdict=approve critical=0 important=0 minor=0 reviewed=17approve. Codex read every changed file, plus.github/workflows/ci.ymlandgo.mod, to confirm nothing builds fromsite/.🤖 Generated with Claude Code
https://claude.ai/code/session_01QQFpfCXs3JqrfyDuiGabF5
Summary by CodeRabbit
Documentation
soar-rsas a wire-compatible Rust client and server.Removals