General text trimming#4
Draft
dolanske wants to merge 4 commits into
Draft
Conversation
zealsprince
reviewed
Jun 15, 2026
zealsprince
reviewed
Jun 15, 2026
|
|
||
| ### Uplink | ||
|
|
||
| Uplink is the IRC layer of an Orbit deployment - an adopted role, not software Orbit builds. Uplink is any stock IRCv3 server; [Ergo](https://ergo.chat/) is the reference implementation, run with no Orbit-specific patches. All persistent communication - messages, channel membership, message history, user identity - flows through Uplink. It is a standard IRCv3 server: no Orbit-specific plugins, no knowledge of Satellite or Depot at the protocol level. Orbit clients and plain IRC clients connect identically. There is no Uplink fork; Orbit runs the server stock and conforms to IRCv3, supporting whatever stock Ergo implements. |
Member
There was a problem hiding this comment.
This is a fairly important line that it's dropping. I'd at least keep the fact that Orbit and normal IRC clients live alongside each other.
zealsprince
reviewed
Jun 15, 2026
| ## What IRC Clients Can Do That Orbit Clients Do | ||
|
|
||
| Everything IRC does well: text chat, channel modes, SASL auth, DMs, bots, ZNC/bouncer support, scripting. An IRC client connecting to Uplink is a first-class participant in the text layer. The spec does not treat IRC clients as second-class. | ||
|
|
Member
There was a problem hiding this comment.
Why would we drop this section? It's fairly important to state the limitations between the clients and make it clear which capabilities are still shared, making existing IRC clients lightweight Orbit clients by default.
zealsprince
requested changes
Jun 15, 2026
zealsprince
left a comment
Member
There was a problem hiding this comment.
Have a few notes for now - otherwise I'm fine with slimming down and trimming line-ending fluff. I was doing a lot of it as opinionated statements trying to drive points home but in your diff I'm realizing I'm doing it more than I remembered.
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
2 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.
As I read the document, I change, trim or edit sections which I think should be simplified. However I think that not all edits I do might be beneficial and could be done better. Treat this PR as a suggestion then. Edits and comments are more than welcome!