andreas / ocaml-graphql-server

GraphQL servers in OCaml
MIT License
624 stars 60 forks source link

Fix websocket conflict, update deprecated Fmt functions #206

Closed zshipko closed 2 years ago

zshipko commented 2 years ago

Fixes #203 by renaming websocket.ml and websocket.mli to graphql_websocket.ml and graphql_websocket.mli

zshipko commented 2 years ago

I don't think the test failure is related to these changes. But I tried adding core_kernel to the graphql-async opam file but that didn't seem to fix the issue.

Edit: Looks like Core_kernel is now deprecated so I just updated the test to use Core instead

andreas commented 2 years ago

Thanks!