Support namespaced keywords

[skuro]

Currently, encoding of keywords strips away the namespace:

user> (yaml/encode {:one/two 42})
;; => {"two" 42}

I discovered this while round-tripping (yaml/parse-string + yaml/generate-string) a Kubernetes YAML, which included something like:

nodeSelector:
    cloud.google.com/gke-nodepool: custom-pool

I fixed it locally with

(extend-type clojure.lang.Keyword
  yaml/YAMLCodec
  (encode [data] (let [n (namespace data)
                       v (name data)]
                   (cond->> v
                     (some? n) (str n "/")))))

[borkdude]

I think this library should support this (by default) and consider this a bug. What do you think @slipset @marcomorain ?

[slipset]

I generally defer to @marcomorain for these kinds of questions, but it seems reasonable to me. @skuro would you be interested in whipping up a PR?

[skuro]

Opened #17 which uses a slightly optimized approach to the same result:

  clojure.lang.Keyword
  (encode [data]
    ;; using clojure.core/name would drop the namespace
    (subs (str data) 1))

Hail hail the anti-if campaign!