-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
build: OwlBot config to specify correct paths with module names #8482
Conversation
- "/google-.*/src/test/java/com/google/cloud/.*/v.*/it/IT.*Test.java" | ||
- "/java-bigquerydatatransfer/google-.*/src/test/java/com/google/cloud/.*/v.*/it/IT.*Test.java" | ||
- "/google-cloud-bigquerydatatransfer/src/main/java/com/google/cloud/bigquery/datatransfer/v1/SampleApp.java" |
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.
This is incorrect.
java-kms/.OwlBot.yaml
Outdated
- "/java-kms/google-.*/src/test/java/com/google/cloud/.*/v.*/it/IT.*Test.java" | ||
- "/java-kms/proto-google-cloud-kms-v1/src/main/java/com/google/cloud/kms/v1/UntypedKeyName.java" | ||
- "/java-kms/proto-google-cloud-kms-v1/src/main/java/com/google/cloud/kms/v1/KeyName.java" | ||
- "/java-kms/proto-google-cloud-kms-v1/src/main/java/com/google/cloud/kms/v1/KeyNames.java" | ||
- "/google-cloud-kms/src/test/java/com/google/cloud/kms/it/ITKmsTest.java" |
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.
This is missed.
java-secretmanager/.OwlBot.yaml
Outdated
- "/google-cloud-secretmanager/src/main/java/com/google/cloud/secretmanager/v1beta1" | ||
- "/google-cloud-secretmanager/src/test/java/com/google/cloud/secretmanager/v1beta1" |
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.
This is missed
java-trace/.OwlBot.yaml
Outdated
- "/google-cloud-trace/src/test/java/com/google/cloud/trace/v1/VPCServiceControlTest.java" | ||
- "/google-cloud-trace/src/test/java/com/google/cloud/trace/v2/VPCServiceControlTest.java" |
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.
These are missed.
java-vision/.OwlBot.yaml
Outdated
- "/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java" | ||
- "/google-cloud-vision/src/test/resources/city.jpg" | ||
- "/google-cloud-vision/src/test/resources/face_no_surprise.jpg" |
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.
These are missed
java-websecurityscanner/.OwlBot.yaml
Outdated
- "/google-cloud-websecurityscanner/src/test/java/com/google/cloud/websecurityscanner/it/v1beta/VPCServiceControlNegativeTest.java" | ||
- "/google-cloud-websecurityscanner/src/test/java/com/google/cloud/websecurityscanner/it/v1beta/VPCServiceControlPositiveTest.java" |
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.
These are missed
…leapis#8482) * build: set_owlbot_config * Updating OwlBot configuration files with modules * build: checking deep-copy destination * /google-cloud entries * fixed google-cloud-* directory entries
Towards #8479