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

Expand on applicability of this scheme for clarity #1806

Merged
merged 2 commits into from
Dec 6, 2021

Conversation

LPardue
Copy link
Contributor

@LPardue LPardue commented Dec 3, 2021

Addressed E#3a of #1802

Reference the HTTP terms we are using. Be clearer that we focus on "normal" responses and that there are other possible uses. But avoid making the entire document a piece of abstract art.

@LPardue LPardue force-pushed the more-definitive-applicability branch from ac6fa37 to ebfa2dc Compare December 3, 2021 03:11
Copy link
Contributor

@kazuho kazuho 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 for the PR. This looks good.

draft-ietf-httpbis-priority.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-priority.md Outdated Show resolved Hide resolved
Co-authored-by: Kazuho Oku <kazuhooku@gmail.com>
@LPardue LPardue mentioned this pull request Dec 6, 2021
30 tasks
@LPardue LPardue merged commit 0c56d3a into main Dec 6, 2021
@LPardue LPardue deleted the more-definitive-applicability branch December 6, 2021 22:33
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.

None yet

2 participants