-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Upgrade to newer Dokka and Knit #2805
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't comment on the changes to the URLs, as I don't understand them. Also, I have only a surface-level understanding of the changes to the build scripts. Hope that's enough.
externalDocumentationLink { | ||
url.set(new URL('http://reactivex.io/RxJava/2.x/javadoc/')) | ||
packageListUrl.set(projectDir.toPath().resolve("package.list").toUri().toURL()) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking through git grep externalDoc
, I notice the lack of Swing and JavaFX modules. It looks like our documentation doesn't reference anything Swing-related, so such a directive is not needed there, but the same is not true for JavaFX.
Also, why does this look different from the way it's done in the reactive module? It looks much more succinct there, and I don't understand the difference between these two modules in this regard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FTR: discussed that reactive module leverages our kts extensions
Regarding JavaFX -- seems to be a general (not PR-related oversight) that should be fixed
Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
# Conflicts: # build.gradle # gradle.properties
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com> (cherry picked from commit bcbcd16)
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com> (cherry picked from commit bcbcd16)
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
No description provided.