-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add Dokka documentation to artifacts #141
Conversation
Codecov Report
@@ Coverage Diff @@
## master #141 +/- ##
============================================
- Coverage 94.73% 94.61% -0.12%
+ Complexity 342 341 -1
============================================
Files 52 52
Lines 892 892
Branches 18 18
============================================
- Hits 845 844 -1
Misses 43 43
- Partials 4 5 +1 |
Vulnerabilities regarding Dokka dependencies can be fixed only by adding dependency constraints in the config repository because they originated from transitive dependencies of the precompiled script dokka-for-java. The vulnerability of error_prone_core is not relevant as this dependency is forced to 2.10.0 which uses 30.1-jre with no mentioned vulnerabilities. |
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.
LGTM.
This PR adds Dokka-generated documentation as a publish artifact.
It is done by using functionality introduced in this PR in config.
Also, while updating the config, the dead code from buildSrc, like the
groovy
directory, was removed.Important details
Breaking changes were introduced in the
2.11.0
version oferror_prone_core
leading to a compilation error. The2.10.0
version was forced for all other Error Prone parts to resolve the issue.