Details
-
Type:
Enhancement
-
Status:
Resolved
-
Priority:
Minor
-
Resolution: Completed
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: None
-
Labels:
-
Environment:clojure/clojurescript "0.0-1450"
-
Patch:Code
Description
The docstrings were parsed from the definitions-forms for def and ns, but not added to the @namespaces metadata map.
There is no :doc entry used in the ns' metadata in the @namespaces.
No ns's :doc info is communicated to the browser in the reflect functions with reflect/doc.
Patch-file is attached with code-changes that add the :doc info for ns and def to the @namespaces, and enhances the reflect functions to communicate that info to the browser in the reflect/doc call.
Attachments
Activity
Frank Siebenlist
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Attachment | add-ns-def-doc-patch.diff [ 11563 ] |
Frank Siebenlist
made changes -
| Attachment | doc-def-ns-reflect.diff [ 11543 ] |
Hide
David Nolen
added a comment -
Excellent, fixed http://github.com/clojure/clojurescript/commit/bef56a74f2eeecabfe0c0a28d89b455dce576ea3
Please at the ticket # to the commit message though, thanks! 
Show
David Nolen
added a comment - Excellent, fixed http://github.com/clojure/clojurescript/commit/bef56a74f2eeecabfe0c0a28d89b455dce576ea3
Please at the ticket # to the commit message though, thanks!
David Nolen
made changes -
| Resolution | Completed [ 1 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] |
This patch no longer applies, mind updating it?