Skip to content

Enhancement: implemented documentation for “codeOnly”-files#115

Open
sjorek wants to merge 2 commits intonevir:masterfrom
sjorek:feature-document-codeonly-files
Open

Enhancement: implemented documentation for “codeOnly”-files#115
sjorek wants to merge 2 commits intonevir:masterfrom
sjorek:feature-document-codeonly-files

Conversation

@sjorek
Copy link
Copy Markdown
Contributor

@sjorek sjorek commented Sep 12, 2013

Description is in #114

@sjorek
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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