Skip to content
This repository has been archived by the owner on Sep 16, 2023. It is now read-only.

docs: remove unused region tags #407

Merged
merged 5 commits into from Jun 3, 2021
Merged

Conversation

Strykrol
Copy link
Contributor

@Strykrol Strykrol commented Mar 1, 2021

Finalizes work started in #325. These tags have proliferated on Devsite; old tags are safe to remove.

@Strykrol Strykrol requested a review from a team March 1, 2021 17:35
@product-auto-label product-auto-label bot added the api: securitycenter Issues related to the googleapis/java-securitycenter API. label Mar 1, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 1, 2021
@snippet-bot
Copy link

snippet-bot bot commented Mar 1, 2021

Here is the summary of changes.

You are about to delete 11 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@codecov
Copy link

codecov bot commented Mar 1, 2021

Codecov Report

Merging #407 (674fb6f) into master (f21f802) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #407   +/-   ##
=========================================
  Coverage     83.63%   83.63%           
  Complexity      548      548           
=========================================
  Files            18       18           
  Lines          4081     4081           
  Branches         62       62           
=========================================
  Hits           3413     3413           
  Misses          598      598           
  Partials         70       70           
Impacted Files Coverage Δ Complexity Δ
...d/securitycenter/v1beta1/SecurityCenterClient.java 84.96% <ø> (ø) 86.00 <0.00> (ø)

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 f21f802...38008cd. Read the comment docs.

@Strykrol
Copy link
Contributor Author

This should be a trivial PR as the new correct region tags are now used in docs. Please merge, thank you for helping with this and related PRs!

@Strykrol
Copy link
Contributor Author

Status here?

@lesv lesv added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 27, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 27, 2021
@lesv
Copy link

lesv commented May 27, 2021

It's failing on windows. I'll merge if you are ok with this / can't merge, we typically review and let you merge.

@lesv
Copy link

lesv commented May 27, 2021

Windows

Warning:  bootstrap class path not set in conjunction with -source 1.7
[INFO] 1 warning
[INFO] -------------------------------------------------------------
[INFO] -------------------------------------------------------------
Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
  symbol:   variable SecurityMarksName
  location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest
[INFO] 1 error


[INFO] ------------------------------------------------------------------------
Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project google-cloud-securitycenter: Compilation failure
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
Error:    symbol:   variable SecurityMarksName
Error:    location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest


Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
  symbol:   variable SecurityMarksName
  location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest
[INFO] 1 error

Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project google-cloud-securitycenter: Compilation failure
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
Error:    symbol:   variable SecurityMarksName
Error:    location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest
Error:  -> [Help 1]
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
Error:  
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <args> -rf :google-cloud-securitycenter

Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
  symbol:   variable SecurityMarksName
  location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest


Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project google-cloud-securitycenter: Compilation failure
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
Error:    symbol:   variable SecurityMarksName
Error:    location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest
Error:  -> [Help 1]


Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
  symbol:   variable SecurityMarksName
  location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest


Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project google-cloud-securitycenter: Compilation failure
Error:  /D:/a/java-securitycenter/java-securitycenter/google-cloud-securitycenter/src/test/java/com/google/cloud/securitycenter/v1beta1/SecurityCenterClientTest.java:[1411,17] cannot find symbol
Error:    symbol:   variable SecurityMarksName
Error:    location: class com.google.cloud.securitycenter.v1beta1.SecurityCenterClientTest
Error:  -> [Help 1]
Error:  
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:  
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
Error:  
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <args> -rf :google-cloud-securitycenter
Error: Process completed with exit code 1.

@lesv
Copy link

lesv commented May 27, 2021

@hannah-tsai PTAL

@Strykrol
Copy link
Contributor Author

@lesv - I don't know why the windows failure would have anything to do with the changes made, to be frank :/ I also do not have write access to this repository, so I was hoping someone could merge it once it's safe. Thanks for responding, let me know if there's anything further I can do here.

@lesv
Copy link

lesv commented May 28, 2021

@Strykrol Np - If you just ask to merge on approval in the comments, I usually do it and would have ping'd earlier. Hopefully @hannah-tsai or someone on her team will take a look on Tuesday - most likely it's something that used to be included is required to be added to the pom.xml files - I'll try rebuilding again - incase it's just something that didn't get read from Maven. If it's not fixed by Wednesday of next week, ping me, and I'll see what we can do.

@lesv lesv added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 28, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 28, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label May 28, 2021
@Strykrol
Copy link
Contributor Author

Oh okay, I can do that moving forward - thank you! We'll check back next week.

@Strykrol
Copy link
Contributor Author

Strykrol commented Jun 3, 2021

@lesv bumping 👍

@lesv lesv merged commit ce983f3 into googleapis:master Jun 3, 2021
@Strykrol
Copy link
Contributor Author

Strykrol commented Jun 3, 2021

Thank you!!

@Strykrol Strykrol deleted the wrapRegionTags branch June 3, 2021 18:08
gcf-merge-on-green bot pushed a commit that referenced this pull request Jun 15, 2021
🤖 I have created a release \*beep\* \*boop\*
---
### [1.6.3](https://www.github.com/googleapis/java-securitycenter/compare/v1.6.2...v1.6.3) (2021-06-15)


### Bug Fixes

* Update dependencies.sh to not break on mac ([#541](https://www.github.com/googleapis/java-securitycenter/issues/541)) ([87a8fef](https://www.github.com/googleapis/java-securitycenter/commit/87a8fefdd6145fd223bedcc12df98dfdb206bcf6))


### Documentation

* remove unused region tags ([#407](https://www.github.com/googleapis/java-securitycenter/issues/407)) ([ce983f3](https://www.github.com/googleapis/java-securitycenter/commit/ce983f33d2492e8db6eca1cb1f357f40f6ceeac7))
* update currently supported Finding filter fields ([#537](https://www.github.com/googleapis/java-securitycenter/issues/537)) ([75eb8a6](https://www.github.com/googleapis/java-securitycenter/commit/75eb8a6e4e68a348e06ed42483a6674367bf6edb))


### Dependencies

* update dependency com.google.cloud:google-cloud-pubsub to v1.113.3 ([#535](https://www.github.com/googleapis/java-securitycenter/issues/535)) ([2cc66ba](https://www.github.com/googleapis/java-securitycenter/commit/2cc66ba13197f0ef95cdad6555c3d46086dab2b5))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v1.3.0 ([#530](https://www.github.com/googleapis/java-securitycenter/issues/530)) ([eb3a2f0](https://www.github.com/googleapis/java-securitycenter/commit/eb3a2f09798b63819ed4e244f6f4b93f5fcc3254))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: securitycenter Issues related to the googleapis/java-securitycenter API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
3 participants