-
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
chore: fix repo-metadata.json api_ids to correctly identify the library coverage #10784
Conversation
a36d3d0
to
eb23ea1
Compare
No region tags are edited in this PR.This comment is generated by snippet-bot.
|
d9b87e7
to
59d6378
Compare
@blakeli0 From the generation action: |
@JoeWang1127 is taking a look. |
@chingor13 Why do you want to change |
|
We have a configuration comparator to determine which libraries are changed and changing For this PR, could you only change |
59d6378
to
91860b1
Compare
I left the |
@@ -208,7 +208,7 @@ Java is a registered trademark of Oracle and/or its affiliates. | |||
[code-of-conduct]: https://github.com/googleapis/google-cloud-java/blob/main/CODE_OF_CONDUCT.md#contributor-code-of-conduct | |||
[license]: https://github.com/googleapis/google-cloud-java/blob/main/LICENSE | |||
[enable-billing]: https://cloud.google.com/apis/docs/getting-started#enabling_billing | |||
[enable-api]: https://console.cloud.google.com/flows/enableapi?apiid=api-gateway.googleapis.com | |||
[enable-api]: https://console.cloud.google.com/flows/enableapi?apiid=apigateway.googleapis.com |
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 changes to the enable API URL look legit and correct to me.
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.
Thanks Jeff! The change LGTM.
Just curious what drove you to make these changes? Are you working on a project to make .repo_metadata.json
up to date? Or only because you noticed something wrong in our .repo_metadata.json
files?
We are not associating these libraries correctly. The
api_id
is the hostname of the API and is used to identify the product/service.