You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's a preference to not need a separate function (e.g. yaml.load vs yaml.loads), but if it's the only way, that's fine.
PR 120 attempts to address this, but I don't like how it designates a file vs a string by the presence of a file header. I think it should be something like if os.exists(filename), then parse file, otherwise try to parse string as raml.
The text was updated successfully, but these errors were encountered:
My preference would be separate functions. The caller should know what they have; a YAML syntax error on what was expected to be a path is likely to confuse an end user.
Like in pyyaml's
yaml.loads
,ramlfication.parse
should be able to take a string of raml format, e.g.There's a preference to not need a separate function (e.g.
yaml.load
vsyaml.loads
), but if it's the only way, that's fine.PR 120 attempts to address this, but I don't like how it designates a file vs a string by the presence of a file header. I think it should be something like if os.exists(filename), then parse file, otherwise try to parse string as raml.
The text was updated successfully, but these errors were encountered: