Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement: document codeOnly-files #114

Open
sjorek opened this issue Sep 12, 2013 · 3 comments
Open

Enhancement: document codeOnly-files #114

sjorek opened this issue Sep 12, 2013 · 3 comments

Comments

@sjorek
Copy link
Contributor

sjorek commented Sep 12, 2013

I'm not yet satisfied with the current codeOnly-implementation, introduced by my own pull-request: #109 . The undocumented code looks good, no question, but having the option to document it by other means would be better.

I'm thinking about placing a *.json.md (obviously containing pure markdown) just besides the *.json files, detecting their availability, and if available use them to render the documentation. Of course I would implement it more generic and take the filename of the current codeOnly-flag enabled file, appending a .md to it and go on as described above. This part would be really easy to implement, _but how to avoid doubled index entries_, if someone also renders any markdown-file by specifying **/*.md. That's the point where I'm stuck and need more research or an idea from someone else …

  :thought_balloon:
😳

@sjorek sjorek mentioned this issue Sep 12, 2013
@kmdavis
Copy link
Collaborator

kmdavis commented Sep 12, 2013

there would also be a tricky part trying to associate documentation blocks with code blocks. it might just be 1 big comment & 1 big code block

@sjorek
Copy link
Contributor Author

sjorek commented Sep 12, 2013

yep … and here we are: #115 ; 1 big comment-block and 1 big code-block, but enough to document the json-files for now … better than no documentation at all … (and no more ugly captions in the index too !)

@sjorek
Copy link
Contributor Author

sjorek commented Sep 12, 2013

latest comments are in the pull-request mentioned above … #115

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants