search

clj-commons / clj-yaml

YAML encoding and decoding for Clojure
Other
122 stars 26 forks source link

Feature/load all #22

Closed clumsyjedi closed 2 years ago

borkdude commented 2 years ago

@clumsyjedi Sorry we didn't get back to you. I've noticed this PR just now. Would you mind cleaning up the changes to project.clj and the .lein_repl_history file?/

/cc @lread

lread commented 2 years ago

Ah. Cool! For those, like me, who would find a description helpful:

A YAML 1.1 stream can contain multiple documents. Currently, clj-yaml will only load the first document. This change adds the ability to optionally load all documents. Enable by specifying :load-all? true in opts for parse-string or parse-stream.

@clumsyjedi thanks for this PR. We realize it's been a while, let me know if you want to finish up the minor changes we need before we can merge it. If we don't hear from you by end of day September 18th, I'll just go ahead and assume we have your blessing.

lread commented 2 years ago

I will carry out recommended changes.