asciidoctor / asciidoclet

:clipboard: A Javadoc Doclet based on Asciidoctor that lets you write Javadoc in the AsciiDoc syntax.
https://github.com/asciidoctor/asciidoclet
Apache License 2.0
133 stars 40 forks source link

Add links to README and Powered-by document. #49

Closed msgilligan closed 8 years ago

msgilligan commented 8 years ago

Addresses:

mojavelinux commented 8 years ago

To follow conventions elsewhere in the project, I think the docs should be in the following folder:

src/docs/asciidoc

(or just src/docs).

I'd also be okay with docs, but it's my third choice.

msgilligan commented 8 years ago

I see your point. Personally, I hate that convention. I like the documentation to be easy to find (even for people who aren't from a development or Java background and aren't used to such deep folder nesting) I also don't see why documentation should be party of the "source code".

But it's not my project, and I do understand the value of conventions. -- I prefer src/docs to src/docs/asciidoc, though -- as little nesting as possible for the docs :)

Tell me where you want it and I'll update the pull request.

mojavelinux commented 8 years ago

@johncarl81 What's your preference?

It needs to at least be "docs" I think (as otherwise it seems like it only contains a single document).

johncarl81 commented 8 years ago

I like src/docs/asciidoc.. follows the maven structure of src/main/java (for instance).

I think we can keep the page discoverable by linking to it from the README. WDYT?

mojavelinux commented 8 years ago

:+1: Good thinking.

msgilligan commented 8 years ago

OK 87011195524124a7fec035ee43bde3e812d332c7 moves the document to src/docs/asciidoc

Note: You'll see I'm using an absolute link from the README. Feel free to change that after the merge. I'd like to learn what the best practice for that is when writing relative links on Github.

mojavelinux commented 8 years ago

I'd like to learn what the best practice for that is when writing relative links on Github.

The best practice is a bit of a hack right now, but here's what I do

= Document Title
ifdef::env-github,env-browser[:outfilesuffix: .adoc]

<<path/to/document.adoc#,label>>
msgilligan commented 8 years ago

Thanks @mojavelinux - relative links are implemented. Pull request should be completely ready to go now.

mojavelinux commented 8 years ago

:+1: Nice job!

msgilligan commented 8 years ago

@johncarl81 Is this one ready to merge? Is there anything else I need to do before it can be accepted?

mojavelinux commented 8 years ago

Looks good to me.

johncarl81 commented 8 years ago

Oops, sorry for the delay...