Visitar URL original
Move <reportPlugins> to <reporting><plugins> by andreamlin · Pull Request #3524 · googleapis/google-cloud-java · GitHub
Skip to content

Move <reportPlugins> to <reporting><plugins> - #3524

Merged
chingor13 merged 8 commits into
googleapis:masterfrom
andreamlin:reporting
Oct 9, 2018
Merged

Move <reportPlugins> to <reporting><plugins>#3524
chingor13 merged 8 commits into
googleapis:masterfrom
andreamlin:reporting

Conversation

@andreamlin

@andreamlin andreamlin commented Aug 1, 2018 •
edited by chingor13
Loading

Copy link
Copy Markdown
Contributor

The build log is cluttered with messages of the sort:

Some problems were encountered while building the effective model for com.google.api.grpc:grpc-google-cloud-automl-v1beta1:jar:0.21.0
[WARNING] Reporting configuration should be done in <reporting> section, not in maven-site-plugin <configuration> as reportPlugins parameter.

Hopefully this shuts it down.

WIP while this is being verified.

Fixes #3205

@andreamlin
andreamlin requested a review from pongad as a code owner August 1, 2018 22:49
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 1, 2018
@andreamlin
andreamlin removed the request for review from pongad August 1, 2018 22:50
@andreamlin andreamlin changed the title Move <reportPlugins> to <reporting><plugins> Move <reportPlugins> to <reporting><plugins> (WIP) Aug 1, 2018
pongad
pongad reviewed Aug 9, 2018
View reviewed changes
Comment thread google-cloud-clients/google-cloud-bigtable/pom.xml Outdated
<properties>
<!-- add this to disable checking -->
<javadoc.opts>-Xdoclint:none</javadoc.opts>
<!--<javadoc.opts>-Xdoclint:none</javadoc.opts>-->

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@pongad

pongad commented Aug 9, 2018

Copy link
Copy Markdown
Contributor

Apart from comment above, LGTM

@andreamlin andreamlin changed the title Move <reportPlugins> to <reporting><plugins> (WIP) Move <reportPlugins> to <reporting><plugins> Aug 14, 2018
@chingor13
chingor13 merged commit ef2a88b into googleapis:master Oct 9, 2018
This was referenced Oct 10, 2018
Closed
Closed
Merged
chingor13 added a commit that referenced this pull request Oct 10, 2018
@chingor13
Revert "Move <reportPlugins> to <reporting><plugins> (#3524)" (#3806)
a66e3b7 
This reverts commit ef2a88b.
@igorbernstein2 igorbernstein2 mentioned this pull request Nov 5, 2018
Closed
suztomo pushed a commit that referenced this pull request Mar 9, 2026
@renovate-bot
deps: update actions/upload-artifact action to v4.4.2 (#3524)
776a554
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
@rahul2393 @cloud-java-bot
docs: add Multi Region Encryption samples (#3524)
7178577 
* docs: add Multi Region Encryption samples

* add create backup sample

* chore: generate libraries at Wed Dec  4 08:33:54 UTC 2024

* add sample for CopyBackup with multi region encryption keys

* add restore backup sample

* fix sample

* chore: generate libraries at Wed Dec  4 09:25:28 UTC 2024

---------

Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
suztomo pushed a commit that referenced this pull request Mar 30, 2026
@andreamlin @chingor13
Move <reportPlugins> to <reporting><plugins> (#3524)
c04cd63 
* move <reportPlugins> to <reporting><plugins>

* update some maven plugins to latest

* remove anchor_build from javadoc_test

* add versions

* comment out javadoc task in bigtable pom

* add build step back to javadoc in circleci

* entirely remove doclint disable in bigtable
suztomo pushed a commit that referenced this pull request Mar 30, 2026
@chingor13
Revert "Move <reportPlugins> to <reporting><plugins> (#3524)" (#3806)
15bda74 
This reverts commit c04cd63.
lqiu96 pushed a commit that referenced this pull request Apr 1, 2026
@andreamlin @chingor13
Move <reportPlugins> to <reporting><plugins> (#3524)
acd9040 
* move <reportPlugins> to <reporting><plugins>

* update some maven plugins to latest

* remove anchor_build from javadoc_test

* add versions

* comment out javadoc task in bigtable pom

* add build step back to javadoc in circleci

* entirely remove doclint disable in bigtable
lqiu96 pushed a commit that referenced this pull request Apr 1, 2026
@chingor13
Revert "Move <reportPlugins> to <reporting><plugins> (#3524)" (#3806)
3b9955a 
This reverts commit c04cd63.
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
@andreamlin @chingor13
Move <reportPlugins> to <reporting><plugins> (googleapis#3524)
8190258 
* move <reportPlugins> to <reporting><plugins>

* update some maven plugins to latest

* remove anchor_build from javadoc_test

* add versions

* comment out javadoc task in bigtable pom

* add build step back to javadoc in circleci

* entirely remove doclint disable in bigtable
benjaminp pushed a commit to benjaminp/google-cloud-java that referenced this pull request Jul 13, 2026
@chingor13
Revert "Move <reportPlugins> to <reporting><plugins> (googleapis#3524)…
e4bfcfd 
…" (googleapis#3806)

This reverts commit 8190258.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@igorbernstein2 igorbernstein2 igorbernstein2 left review comments
@chingor13 chingor13 chingor13 approved these changes
+1 more reviewer
@pongad pongad pongad left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@andreamlin @pongad @chingor13 @igorbernstein2 @googlebot