build(deps): remove dokka, no longer in use #17621
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose / Description
It doesn't appear that we actually use dokka anymore, remove the dependency
Fixes
Approach
Remove all references to dokka in our build files
How Has This Been Tested?
We use jitpack.io to publish the API artifacts, and in our
jitpack.yml
we run./gradlew :api:publishToMavenLocal
Anki-Android/jitpack.yml
Line 15 in ff607b3
So I ran that then checked the javadoc artifact (attached) - it does generate (which is job number one) and it appears fine.
To be clear though, I'm not that interested in the output, it's mainly generated at all because some publishers (maven at least, jitpack unknown) require you to have a javadoc jar.
But we also publish the actual sources of the API module since we are open source, and most (all?) IDEs will auto-pull those and cross-link everything up and let you just open things and look. Which is 100 times more useful than javadoc and the primary reason why I haven't actually looked at anyone's javadoc in ages.
api-2.0.0-javadoc.zip
Learning (optional, can help others)
I'd almost forgotten that javadoc existed, but then I don't work much with closed-source products.
Checklist
Please, go through these checks before submitting the PR.