Skip to content

Address Editorial Feedback in #218 #624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Address Editorial Feedback in #218 #624

merged 3 commits into from
Dec 10, 2024

Conversation

afrind
Copy link
Collaborator

@afrind afrind commented Nov 15, 2024

One small normative change: I relaxed a SHOULD to a MAY when discussing relays writing data out of order. No one is implementing this right now.

Fixes #218

One small normative change: I relaxed a SHOULD to a MAY when discussing relays writing data out of order.  No one is implementing this right now.

Fixes #218
@@ -1089,11 +1087,11 @@ handles a subscription that includes those Objects re-requests them.
## CLIENT_SETUP and SERVER_SETUP {#message-setup}

The `CLIENT_SETUP` and `SERVER_SETUP` messages are the first messages exchanged
by the client and the server; they allows the peers to establish the mutually
by the client and the server; they allows the endpoints to establish the mutually
Copy link
Collaborator

Choose a reason for hiding this comment

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

s/allows/allow

@ianswett ianswett added the Editorial The draft is difficult to understand on a specific point, or it is open to multiple interpretations. label Nov 19, 2024
Copy link
Collaborator Author

@afrind afrind left a comment

Choose a reason for hiding this comment

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

I'm tidying this to clear the issue backlog, but I'm not sure any of this belongs in MoQT anymore. It probably belongs in an overview document.

@ianswett ianswett merged commit c6096b8 into main Dec 10, 2024
2 checks passed
@englishm englishm mentioned this pull request Feb 25, 2025
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial The draft is difficult to understand on a specific point, or it is open to multiple interpretations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Editorial notes
3 participants