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

Refresh the README for modern developments #682

Merged
merged 1 commit into from
May 26, 2022
Merged

Conversation

rossabaker
Copy link
Member

No description provided.

@rossabaker rossabaker changed the base branch from main to series/0.23 May 26, 2022 02:36
Copy link
Member

@danicheg danicheg left a comment

Choose a reason for hiding this comment

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

Yesterday, I also was thinking about readme changing. It's fantastic that you have done it!

v0.23.11. They are now maintained alongside the blaze core. These
modules depend on `http4s-core` and `blaze-http`.

* [`http4s-blaze-core`][./blaze-core]: common, functional code to in
Copy link
Member

Choose a reason for hiding this comment

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

I'm not a native, but probably that 'to' is redundant

Suggested change
* [`http4s-blaze-core`][./blaze-core]: common, functional code to in
* [`http4s-blaze-core`][./blaze-core]: common, functional code in

- Type safe pipeline composition
- NIO1 and NIO2 socket engines
- SSL integration
- HTTP/1.x and HTTP2 codecs
- HTTP/1.x and HTT/P2 codecs
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- HTTP/1.x and HTT/P2 codecs
- HTTP/1.x and HTTP/2 codecs

@rossabaker rossabaker merged commit 0fc26e1 into series/0.23 May 26, 2022
@rossabaker rossabaker deleted the refresh-readme branch October 24, 2024 17:34
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