jbt / docker

Documentation generator
http://jbt.github.com/docker
MIT License
234 stars 56 forks source link

Big update (v0.3 / 1.0?) #75

Closed jbt closed 8 years ago

jbt commented 9 years ago

I don't have any code to show for this yet because my experiments are too messy, but I'm interested to know people's opinions on the following propositions for the next major version:

If anyone has any thoughts or suggestions on this, or thinks any of these propositions are a Really Bad Idea, then do please reply.

jasonseney commented 9 years ago

Overall, I think this is a good idea for the direction of the project. Like you mentioned already, there will be a big initial hit in language support. The list of languages can be viewed and compared here:

Personally, for my own use, Highlights would be better since it supports Sass, which is missing in Pygments. Looking at the list though, I do notice it's missing some template languages and notably C#. I've heard that Atom can get weird with certain short tags, but haven't used it enough myself to say how specifically.

I think we should keep this open for discussion but might be a little too soon to make the switch. Perhaps a separate branch would be nice to start working on it though and try out some languages.

As for the modules, +1!

ozum commented 9 years ago

+1 for using other modules. +1 for using highlights

Extra feature suggestions:

I may suggest changes below for comments located same line. EDIT: This messes up other things, since I don't know inner working of the module exactly.

docker.js Line 499:
--- var commentRegex = new RegExp('^\\s*' + params.comment + '\\s?');
+++ var commentRegex = new RegExp('\\s*' + params.comment + '\\s?');

docker.js Line 592:
--- section.docs += line.replace(commentRegex, '') + '\n';
+++ var result = new RegExp('(.*)?\\s*' + params.comment + '\\s?(.*)?').exec(line);
+++ section.code += result[1] + '\n';
+++ section.docs += result[2] + '\n';

Thank you,

chiefjester commented 9 years ago

hey @jbt looking forward to this update.