-
Notifications
You must be signed in to change notification settings - Fork 1
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
Merging bsb-http-chunked into blaze-builder? #1
Comments
I am just a bound-bumper here, really, maybe @jaspervdj has more specific opinions. |
Simon's vision was that this code should really be part of ByteString, and I still think this is the right approach. However, after the code was ported to ByteString, this package sort of stuck around, mostly due to gaps in the API, like @sjakobi pointed out. Having an additional package |
Alright. If the goal still is to eventually retire Thanks for the feedback! :) |
I extracted the
bsb-http-chunked
package fromblaze-builder
back whenblaze-builder
seemed largely unmaintained and in demise.warp
now usesbsb-http-chunked
, buthttp-streams
andsnap-server
have stuck withblaze-builder
so far.Now that
blaze-builder
is actively maintained again, it seems pretty pointless to maintain the same API in two places.The main changes that I expect I could contribute to
Blaze.ByteString.Builder.HTTP
are:I'm not very keen to be involved in general package maintenance, but I'd be willing to help with issues specific to
Blaze.ByteString.Builder.HTTP
.What do you think?
The text was updated successfully, but these errors were encountered: