-
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
feat: [cloudasset] added new resource references to fields in AnalyzeMoveRequest #10138
Conversation
feat: added Asset.access_policy, access_level, service_perimeter, org_policy feat: added resource definitions to some messages docs: updated comments chore: removed backend configuration from service config PiperOrigin-RevId: 589961470 Source-Link: googleapis/googleapis@02fbe75 Source-Link: https://github.com/googleapis/googleapis-gen/commit/2a25ee3638abdc2ea520c87e1d4c7c53fc624e30 Copy-Tag: eyJwIjoiamF2YS1hc3NldC8uT3dsQm90LnlhbWwiLCJoIjoiMmEyNWVlMzYzOGFiZGMyZWE1MjBjODdlMWQ0YzdjNTNmYzYyNGUzMCJ9
docs: updated comments chore: removed backend configuration from service config PiperOrigin-RevId: 590982722 Source-Link: googleapis/googleapis@da09f4c Source-Link: https://github.com/googleapis/googleapis-gen/commit/7dd789a2cad4d544fbb53a15ce59deb467f30b6d Copy-Tag: eyJwIjoiamF2YS1hc3NldC8uT3dsQm90LnlhbWwiLCJoIjoiN2RkNzg5YTJjYWQ0ZDU0NGZiYjUzYTE1Y2U1OWRlYjQ2N2YzMGI2ZCJ9
No region tags are edited in this PR.This comment is generated by snippet-bot.
|
@diegomarquezp What needs to happen here?
|
Missing dependencies?
Why What does the problematic line look like?
Where is How to reproduce the issue
|
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.
Unit tests passed
<dependency> | ||
<groupId>com.google.api.grpc</groupId> | ||
<artifactId>proto-google-cloud-orgpolicy-v1</artifactId> | ||
</dependency> | ||
<dependency> | ||
<groupId>com.google.api.grpc</groupId> | ||
<artifactId>proto-google-identity-accesscontextmanager-v1</artifactId> | ||
</dependency> | ||
</dependencies> |
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.
docs: updated comments
chore: removed backend configuration from service config
PiperOrigin-RevId: 590982722
Source-Link: googleapis/googleapis@da09f4c
Source-Link: https://github.com/googleapis/googleapis-gen/commit/7dd789a2cad4d544fbb53a15ce59deb467f30b6d
Copy-Tag: eyJwIjoiamF2YS1hc3NldC8uT3dsQm90LnlhbWwiLCJoIjoiN2RkNzg5YTJjYWQ0ZDU0NGZiYjUzYTE1Y2U1OWRlYjQ2N2YzMGI2ZCJ9
BEGIN_NESTED_COMMIT
feat: [cloudasset] added messages ExportAssetsResponse, BatchGetAssetsHistoryResponse
feat: added Asset.access_policy, access_level, service_perimeter, org_policy
feat: added resource definitions to some messages
docs: updated comments
chore: removed backend configuration from service config
PiperOrigin-RevId: 589961470
Source-Link: googleapis/googleapis@02fbe75
Source-Link: https://github.com/googleapis/googleapis-gen/commit/2a25ee3638abdc2ea520c87e1d4c7c53fc624e30
Copy-Tag: eyJwIjoiamF2YS1hc3NldC8uT3dsQm90LnlhbWwiLCJoIjoiMmEyNWVlMzYzOGFiZGMyZWE1MjBjODdlMWQ0YzdjNTNmYzYyNGUzMCJ9
END_NESTED_COMMIT