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

Process descriptions for @link tags. #79

Merged
merged 1 commit into from Jan 12, 2015

Conversation

nicksay
Copy link
Contributor

@nicksay nicksay commented Dec 24, 2014

Symbols within the same AST are linked via generated hash targets. Absolute
URLs starting with http:// and https:// are linked normally. Any other
values are not linked and output as text.

Note: Cross-AST (aka multi-file) symbol linking could be achieved in the future
by creating a global symbol map before generating the output.

Symbols within the same AST are linked via generated hash targets.  Absolute
URLs starting with `http://` and `https://` are linked normally.  Any other
values are not linked and output as text.

Note: Cross-AST (aka multi-file) symbol linking could be achieved in the future
by creating a global symbol map before generating the output.
@nicksay
Copy link
Contributor Author

nicksay commented Dec 24, 2014

This change is useful for generating a same-page index with links to symbols. Here's an example:
http://youtube.github.io/spfjs/api/

Let me know what you think!

@mrjoelkemp
Copy link
Contributor

Looks great @nicksay. Nice job! +1 from me.

I originally thought this would be done in lib/analyze, but it would be redundant operation across many types. Putting it in lib/generateMD is quite convenient.

Thanks for contributing.

@nicksay
Copy link
Contributor Author

nicksay commented Jan 12, 2015

Thanks @mrjoelkemp. @psq any thoughts?

@psq
Copy link
Contributor

psq commented Jan 12, 2015

sorry guys, somehow I totally missed this PR. So thanks for the gentle ping, @nicksay. Looks quite nice. Will merge and release.

psq added a commit that referenced this pull request Jan 12, 2015
@psq psq merged commit 21e9250 into sutoiku:master Jan 12, 2015
@nicksay
Copy link
Contributor Author

nicksay commented Jan 12, 2015

Thanks!

@nicksay nicksay deleted the process-descriptions branch January 12, 2015 23:07
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