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

[nesting] path construction in comment parser #73

Closed
wants to merge 1 commit into from

Conversation

secretrobotron
Copy link

  1. Implemented simple path construction (adjustPath) in parseComments.
  2. Added test.
  3. Added a fixture for said test.

Without disrupting the way dox already constructs its output, this patch offers a way for people to use the output to build nested structure. For example, having a class with members: paths offer some insight into how your documentation can be structured.

1. Implemented simple path construction (adjustPath) in parseComments.
2. Added test.
3. Added a fixture for said test.
@tj
Copy link
Owner

tj commented Nov 22, 2012

hmm im slightly confused by this, why nest them all weird?

@secretrobotron
Copy link
Author

I'm just trying to automate our documentation system a little bit. Knowing what's parented by what else, I can structure documentation output (html or something) in such a way that member functions can be nested inside classes, etc.

Is there another way to accomplish this? I did write this in the morning...

@tj
Copy link
Owner

tj commented Nov 22, 2012

I just haven't seen them nested like that, maybe it's best in a fork of dox

@secretrobotron
Copy link
Author

Which part is confusing though? Nesting like https://github.com/visionmedia/dox/pull/73/files#L2R27 ?

@ouhouhsami ouhouhsami mentioned this pull request Sep 4, 2014
@evindor
Copy link
Collaborator

evindor commented Sep 7, 2014

Outdated, sorry.

@evindor evindor closed this Sep 7, 2014
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.

None yet

3 participants