Closed ysangkok closed 1 year ago
Thanks @ysangkok for the PR. I'll take some time between this and next week to give it try.
@felixminom Much appreciated! Should I submit similar PRs for req and modern-uri?
That's something I will do anyway, but right now the upgrade is blocked on weigh
.
@mrkkrp Hmm, how is it blocked on weigh
? I don't see it in the cabal-plan topo
listing. Sorry for being dense.
Ooh, I see it now in the benchmark for modern-uri: https://github.com/mrkkrp/modern-uri/blob/b6161a81dbc04af5fa9ae96bd0d6bb23a96ddfdd/modern-uri.cabal#L127
PR created:
Thanks for the follow-up. I'll keep an eye on this and merge once weigh
's PR is ready.
@felixminom weigh-0.0.17 was released with the necessary changes
@ysangkok thank you, just merged another PR. Will merge this as well and prepare a release.
Fixes #84