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

Add an article about multiplexing #465

Merged
merged 3 commits into from
Sep 18, 2024
Merged

Conversation

glbrntt
Copy link
Contributor

@glbrntt glbrntt commented Sep 18, 2024

Motivation:

It's hard to keep track of all the pieces that do multiplexing. When making some changes I found myself writing down what each piece did and how they fit together. I think this is generally useful information for anyone maintaining the library.

Modifications:

  • Add a DocC article about the different multiplexing approaches that's aimed at maintainers of NIOHTTP2.

Result:

Easier to learn about how multiplexing is done

Motivation:

It's hard to keep track of all the pieces that do multiplexing. When
making some changes I found myself writing down what each piece did and
how they fit together. I think this is generally useful information for
anyone maintaining the library.

Modifications:

- Add a DocC article about the different multiplexing approaches that's
  aimed at _maintainers_ of NIOHTTP2.

Result:

Easier to learn about how multiplexing is done
@glbrntt glbrntt added the semver/none No version bump required. label Sep 18, 2024
@glbrntt glbrntt changed the title Add an articles about multiplexing Add an article about multiplexing Sep 18, 2024
Copy link
Contributor

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

Mostly looks great, one optional note

…rs.md

Co-authored-by: Cory Benfield <lukasa@apple.com>
@glbrntt glbrntt enabled auto-merge (squash) September 18, 2024 17:21
@glbrntt glbrntt merged commit eaa71bb into apple:main Sep 18, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants