Skip to content
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

Editorial changes mainly for style guide alignment #2658

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Conversation

LPardue
Copy link
Contributor

@LPardue LPardue commented Oct 19, 2023

See https://httpwg.org/admin/editors/style-guide.

Lots of boring editorial tweaks to ensure better consistency both
within the document and other HTTP specs.

See https://httpwg.org/admin/editors/style-guide.

Lots of boring editorial tweaks to ensure better consistency both
within the document and other HTTP specs.
@LPardue LPardue changed the title Editorial changes mainly for style guide alignement Editorial changes mainly for style guide alignment Oct 19, 2023
Copy link
Member

@Acconut Acconut left a comment

Choose a reason for hiding this comment

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

Thank you very for this tedious work! I only spotted a few minor typos.

draft-ietf-httpbis-resumable-upload.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-resumable-upload.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-resumable-upload.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-resumable-upload.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-resumable-upload.md Outdated Show resolved Hide resolved
@LPardue LPardue merged commit c6a88b9 into main Oct 19, 2023
@LPardue LPardue deleted the lucas-has-nits branch October 19, 2023 13:55
@Acconut
Copy link
Member

Acconut commented Oct 20, 2023

One related question: The style guide also mentions that "examples should be in HTTP/1.1 format unless they are specific to another version of the protocol", but the draft uses a notation that resembles HTTP/2. Should we change the example messages to HTTP/1.1?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants