Skip to content

conduit-async: fix bounds for async#12554

Merged
mseri merged 3 commits into
ocaml:masterfrom
samoht:cohttp-async
Sep 4, 2018
Merged

conduit-async: fix bounds for async#12554
mseri merged 3 commits into
ocaml:masterfrom
samoht:cohttp-async

Conversation

@samoht
Copy link
Copy Markdown
Member

@samoht samoht commented Sep 3, 2018

Should fix the issue found in #12552

@camelus
Copy link
Copy Markdown
Contributor

camelus commented Sep 3, 2018

✅ All lint checks passed 5a7ec80
  • These packages passed lint tests: cohttp-async.1.0.0, cohttp-async.1.0.2, horned_worm.0.3.2

✅ Installability check (9301 → 9301)

@samoht
Copy link
Copy Markdown
Member Author

samoht commented Sep 3, 2018

I'm moved some of the revdeps fixes to #12556 to avoid saturating the CI..

@mseri mseri closed this Sep 4, 2018
@mseri mseri reopened this Sep 4, 2018
@mseri
Copy link
Copy Markdown
Member

mseri commented Sep 4, 2018

Let’s re run the tests with the sedlex and flow fixes

@samoht
Copy link
Copy Markdown
Member Author

samoht commented Sep 4, 2018

Rebased

@mseri
Copy link
Copy Markdown
Member

mseri commented Sep 4, 2018

Travis failures are few and all timeouts due to the high number of packages built and tested.
The CI failure are all infrastructure related, so I think we can merge

@samoht
Copy link
Copy Markdown
Member Author

samoht commented Sep 4, 2018

so I think we can merge

sounds good to me :-)

@mseri mseri merged commit 6b356cb into ocaml:master Sep 4, 2018
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.

3 participants