Closed ndarilek closed 4 years ago
Merged! :)
I published both the crates, too. Seems to be working fine. Thanks so much for working on this--I don't think the project would've got anywhere otherwise. Barring major Tolk upgrades, I don't think there need to be many more changes (although if you think of any, please feel free to send a PR and I'll publish as soon as possible).
Good luck on your future projects!
Excellent, thanks! I might eventually remove the method syntax since Tolk itself seems to be a singleton, but I'll save that for a 0.3 in the future.
These commits do a number of things:
Option<String>
to handle the null case, which currently crashes consumers.output
. Let's stick with the library's behavior on this one.Into<String>
everywhere&str
was expected before, thus allowing any string-like value.I probably missed a few things, but the example works on my system, and I'll be actively using this crate for the time being. So if anything was missed, I'll attempt a fix and push out a change.
Thanks!