flyx / NimYAML

YAML implementation for Nim
https://nimyaml.org
Other
186 stars 36 forks source link

Export yaml specific exceptions that are part of API #144

Closed kdeme closed 1 month ago

kdeme commented 1 month ago

YamlConstructionError and YamlParserError are part of the function signatures in those modules and thus they should be re-exported imo. Else you need to import specific modules to be able to access them.

flyx commented 1 month ago

You're absolutely correct. I'll merge this disregarding the failed tests, that's just upstream Nim devel breaking stuff. I've seen it before :)

Thanks!

kdeme commented 1 month ago

Great, thanks for the quick merge.