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

HTTP/2 server example #139

Merged
merged 6 commits into from
Sep 23, 2024
Merged

HTTP/2 server example #139

merged 6 commits into from
Sep 23, 2024

Conversation

n1haldev
Copy link
Contributor

This adds http2 server example to the website as mentioned in hyperium/hyper#3568

@seanmonstar
Copy link
Member

Thanks for improving the documentation! I was thinking, since it's so similar to the "hello world" guide, that perhaps what might be best is to just update that guide. Where it links out to the example, add a few words "(and a similar example using HTTP/2)". What do you think?

@n1haldev
Copy link
Contributor Author

Ya I think that is good. I will work on that.

@n1haldev
Copy link
Contributor Author

@seanmonstar I have made the requested changes to this PR as per your recommendations. Just wanted to know if there is any other change to be implemented or if the PR is ready for merging.
PS : This other PR that I have raised, hyperium/hyper#3702, is also related to the same issue.

Copy link
Member

@seanmonstar seanmonstar left a comment

Choose a reason for hiding this comment

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

Thanks for reminding me about the example. I'm glad you're helping make it easier for people to learn how to use hyper! <3

I had some thoughts on adjusting slightly where this content goes, and then I think we'd be ready to go. Thank you again!

_stable/server/hello-world.md Show resolved Hide resolved
_stable/server/hello-world.md Outdated Show resolved Hide resolved
@n1haldev
Copy link
Contributor Author

I have implemented the necessary changes.

Copy link
Member

@seanmonstar seanmonstar left a comment

Choose a reason for hiding this comment

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

Beautiful, thank you!

@seanmonstar seanmonstar merged commit 62a6f66 into hyperium:master Sep 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants