edn spec does not yet include namespace map syntax #78
Comments
Closed
iarenaza
added a commit
to iarenaza/parseedn
that referenced
this issue
Sep 8, 2023
parseclj already added support for namespaced maps in 2018 (in commit b40670a56147214f0486763529897cb688a09692). As Alex Miller said in edn-format/edn#78 > Clojure introduced namespace map syntax in Clojure 1.9. The Clojure > edn reader was also updated to support the non-autoresolved parts of > namespace map syntax > > The edn spec should be updated to a new version that includes the > namespace map syntax such as #:foo{:bar 1} (syntax alternative for > {:foo/bar 1}). So it's pretty clear that the intention is for edn spec to support namespaced maps (the reference implementation in Clojure already does!). In addition to the use case mentioned in issue clojure-emacs#16, not supporting this seems to cause trouble in CIDER when integrating with shadow-cljs (clojure-emacs/cider#3437 Fixes: clojure-emacs#16
iarenaza
added a commit
to iarenaza/parseedn
that referenced
this issue
Sep 9, 2023
parseclj already added support for namespaced maps in 2018 (in commit b40670a56147214f0486763529897cb688a09692). As Alex Miller said in edn-format/edn#78 > Clojure introduced namespace map syntax in Clojure 1.9. The Clojure > edn reader was also updated to support the non-autoresolved parts of > namespace map syntax (edn doesn't do anything autoresolved like > ::foo or #::foo{}). > > The edn spec should be updated to a new version that includes the > namespace map syntax such as #:foo{:bar 1} (syntax alternative for > {:foo/bar 1}). So it's pretty clear that the intention is for edn spec to support namespaced maps (the reference implementation in Clojure already does!). In addition to the use case mentioned in issue clojure-emacs#16, not supporting this seems to cause trouble in CIDER when integrating with shadow-cljs (clojure-emacs/cider#3437) Fixes: clojure-emacs#16
iarenaza
added a commit
to iarenaza/parseedn
that referenced
this issue
Sep 11, 2023
parseclj already added support for namespaced maps in 2018 (in commit b40670a56147214f0486763529897cb688a09692). As Alex Miller said in edn-format/edn#78 > Clojure introduced namespace map syntax in Clojure 1.9. The Clojure > edn reader was also updated to support the non-autoresolved parts of > namespace map syntax (edn doesn't do anything autoresolved like > ::foo or #::foo{}). > > The edn spec should be updated to a new version that includes the > namespace map syntax such as #:foo{:bar 1} (syntax alternative for > {:foo/bar 1}). So it's pretty clear that the intention is for edn spec to support namespaced maps (the reference implementation in Clojure already does!). In addition to the use case mentioned in issue clojure-emacs#16, not supporting this seems to cause trouble in CIDER when integrating with shadow-cljs (clojure-emacs/cider#3437) Fixes: clojure-emacs#16
vemv
pushed a commit
to clojure-emacs/parseedn
that referenced
this issue
Sep 29, 2023
parseclj already added support for namespaced maps in 2018 (in commit b40670a56147214f0486763529897cb688a09692). As Alex Miller said in edn-format/edn#78 > Clojure introduced namespace map syntax in Clojure 1.9. The Clojure > edn reader was also updated to support the non-autoresolved parts of > namespace map syntax (edn doesn't do anything autoresolved like > ::foo or #::foo{}). > > The edn spec should be updated to a new version that includes the > namespace map syntax such as #:foo{:bar 1} (syntax alternative for > {:foo/bar 1}). So it's pretty clear that the intention is for edn spec to support namespaced maps (the reference implementation in Clojure already does!). In addition to the use case mentioned in issue #16, not supporting this seems to cause trouble in CIDER when integrating with shadow-cljs (clojure-emacs/cider#3437) Fixes: #16
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Clojure introduced namespace map syntax in Clojure 1.9. The Clojure edn reader was also updated to support the non-autoresolved parts of namespace map syntax (edn doesn't do anything autoresolved like
::fooor#::foo{}).The edn spec should be updated to a new version that includes the namespace map syntax such as
#:foo{:bar 1}(syntax alternative for{:foo/bar 1}).The text was updated successfully, but these errors were encountered: