<@U0AAQKT9Q> I’ve tried but no luck yet. See <http...
# build-tools
h
i
Does the output of
dokka
task give any clue?
I suppose you need to configure that task a little bit beyond minimal configuration, namely specify where to look for samples.
h
So is it not considering gradle’s source sets as a default?
Thanks @ilya.gorbunov for your help. First thing I’ve noticed is that it is not cleaning generated docs on “gradle clean” (not sure if it should?!). So to run it again, I always have to clean manually. Second, concerning the dokka output: it output does not indicate any issues with my
@sample
in the gradle logs when running the
dokka
task Third tooling could be better, that is in Intellij it would be great to be able to jump to the referenced code bit by clicking on it (mockup:

https://i.imgur.com/d8VtklB.png

). By doing so, missing/incorrect references could be highlighted in red. Feel welcome to point me to any tickets I could vote for/comment on.
o
@semoro ^^
i
@holgerbrandl I've checked out your project and played with it a bit. First, note that multiline doc comments should have
*
on each line, otherwise they are not parsed correctly in IDE:
Copy code
/**
 * @sample krangl.samples.buildDataFrameof
 */
Actually Ctrl+Click starts working as soon as you're using this format.
Second, it may be not a good idea to place samples in the production source root, try to place them in the tests source set, or even in some custom source set. Then you'll need to specify path to samples in dokka configuration.
h
Indeed prefixing javadoc with * enables tooling. Works like charm now! 🙂 The other issue was that I used “Copy Reference” in Intellij to obtain the reference to the sample method (which is just a package level function and not a class member function). However this gives
krangl.samples.DokkaSamplesKt.buildDataFrameof
instead of the expected/correct
krangl.samples.buildDataFrameof
. I guess thats more a kotlin plugin bug, not sure if there’s a ticket for it. For sure the samples should not sit in the production root, but I wanted a working setup before moving them around. Now they are part of test. Thanks a lot for your help.
o
Please file an issue for “Copy Reference”
h