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: implemented documentation for “codeOnly”-files #115

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sjorek
Copy link
Contributor

@sjorek sjorek commented Sep 12, 2013

Description is in #114

@sjorek
Copy link
Contributor Author

sjorek commented Sep 12, 2013

Dunno with thread you read first, so here is a comment duplicate to ease reading 😏

yep … and here we are: 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

If you decide to merge this, you could leave the issue and/or this request open, so we can discuss a solution for how to sync code and comments (= having more than 1 comment-block and 1 code-block) …

… I have something like the typical unified-patch-diff-syntax in mind … the section about “range information”

@sjorek
Copy link
Contributor Author

sjorek commented Sep 12, 2013

… another idea to get code and comments in sync: place regular-expressions as markers in the markdown-file to indicate what line of code the comment-segment should start at … sounds easier than the one above … an example:

code:

{
 "a nifty and nice": "json file",
 "with numbers": 1,
 "and booleans: true
}

markdown:

# description of the file

All lines starting and ending with / will be stripped from output and used
as regular expression to match the line of code the following comment
segment should start at. If it does not match at all, the code and comments
being left will be append to the previous segment.

/a nifty/
This parameter is the first property to describe.

/with numbers/
This parameter is the second property to describe.

/and booleans/
This parameter is the third property to describe.

/does not match/
… this will be appended to the segment above.

… actually I like this idea, as I'm writing it down. @kmdavis How about you ?

@sjorek
Copy link
Contributor Author

sjorek commented Sep 12, 2013

Attention: this patch should have #116 applied ! Otherwise the nicely documented .groc.json file will be missing on github !

You did it, yeah !

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

Successfully merging this pull request may close these issues.

1 participant