dbuenzli / fmt

OCaml Format pretty-printer combinators
http://erratique.ch/software/fmt
ISC License
71 stars 26 forks source link

Add Fmt.{failwith, invalid_arg} #25

Closed hcarty closed 7 years ago

hcarty commented 7 years ago

Closes #24

hcarty commented 7 years ago

I wasn't sure how terse to keep the doc comments - please let me know if you'd prefer just the [failwith] is [kstrf failwith] text as the comment string.

dbuenzli commented 7 years ago

Thanks your patch is in as eb07861d2a5292e7a16ff97, I slightly tweaked the doc string in a2d065bfc555264edf688c6.

hcarty commented 7 years ago

Thank you! The tweaked doc strings are definitely an improvement.

hcarty commented 7 years ago

And apologies for not including a changes entry - I completely missed that.

dbuenzli commented 7 years ago

And apologies for not including a changes entry - I completely missed that.

Bah np the commit is sufficently clear so topkg status will make that a breeze.