KEMBAR78
chore: fix javadoc site after repo split by igorbernstein2 · Pull Request #89 · googleapis/java-bigtable · GitHub
Skip to content

Conversation

@igorbernstein2
Copy link
Contributor

  • To ensure that the excludes are respected by both the aggregate site and the javadoc jars
    • move the javadoc config from reporting to build configs: this is necessary because the javadoc:jar is run as part of the build phase
    • move the javadoc config to the parent pom so that it can be used by both the aggregate site goal and the javadoc jars
  • Exclude the grpc & protobuf stubs from the aggregate site: they are not part of the public api in the java client
  • Remove unused external links
  • Stop generating unused site reports, we only use the javadocs

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for context and/or discussion)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 15, 2019
@igorbernstein2 igorbernstein2 changed the title chore: Fix javadoc site after repo split chore: fix javadoc site after repo split Nov 15, 2019
* To ensure that the excludes are respected by both the aggregate site and the javadoc jars
  * move the javadoc config from reporting to build configs: this is necessary because the javadoc:jar is run as part of the build phase
  * move the javadoc config to the parent pom so that it can be used by both the aggregate site goal and the javadoc jars
* Exclude the grpc & protobuf stubs from the aggregate site: they are not part of the public api in the java client
* Remove unused external links
* Stop generating unused site reports, we only use the javadocs
@codecov
Copy link

codecov bot commented Nov 15, 2019

Codecov Report

Merging #89 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #89      +/-   ##
============================================
- Coverage     81.31%   81.07%   -0.24%     
  Complexity      937      937              
============================================
  Files            95       95              
  Lines          5849     5849              
  Branches        325      311      -14     
============================================
- Hits           4756     4742      -14     
+ Misses          917      916       -1     
- Partials        176      191      +15
Impacted Files Coverage Δ Complexity Δ
...igtable/admin/v2/BaseBigtableTableAdminClient.java 56.78% <0%> (-4.03%) 48% <0%> (ø)
...able/admin/v2/BaseBigtableInstanceAdminClient.java 59.16% <0%> (-3.15%) 54% <0%> (ø)
...om/google/cloud/bigtable/emulator/v2/Emulator.java 60% <0%> (ø) 14% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 83ac4c8...417543c. Read the comment docs.

@igorbernstein2 igorbernstein2 merged commit b3da95a into googleapis:master Nov 15, 2019
@igorbernstein2 igorbernstein2 deleted the fix-javadoc branch November 15, 2019 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants