hyper-systems / ocaml-mqtt

MQTT client for OCaml
https://hyper-systems.github.io/ocaml-mqtt
19 stars 5 forks source link

Assorted fixes #2

Closed ulrikstrid closed 3 years ago

ulrikstrid commented 3 years ago

I'm trying to use this in a small service in production and these are changes I find while setting it up.

I can split it out into multiple PRs later if you want to.

rizo commented 3 years ago

Hey! Happy to hear you're using this library. Let me know if you need any help or find any issues.

Thanks for all the corrections. No need to split them into separate PRs (unless you prefer to).

rizo commented 3 years ago

@ulrikstrid Let me know when the PR is ready for review. I can it's currently set to "draft".

ulrikstrid commented 3 years ago

It's ok to review now @rizo we're happily running it on production 👍

rizo commented 3 years ago

Thank you!