Skip to content

docs(maps): correct the samples#14632

Merged
bhshkh merged 1 commit into
googleapis:mainfrom
bhshkh:chore/maps-samples
May 22, 2026
Merged

docs(maps): correct the samples#14632
bhshkh merged 1 commit into
googleapis:mainfrom
bhshkh:chore/maps-samples

Conversation

@bhshkh
Copy link
Copy Markdown
Contributor

@bhshkh bhshkh commented May 22, 2026

Fixes: b/421113149

@bhshkh bhshkh requested review from a team as code owners May 22, 2026 16:01
@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label May 22, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the librarian.yaml configuration to omit snippets for several Maps APIs and modifies the maps/README.md to update the code samples link. A review comment suggests that the new link in the README is too specific for the entire module and recommends providing both the general and specific sample links to ensure all users can find relevant resources.

Comment thread maps/README.md
## Google Cloud Samples

To browse ready to use code samples check [Google Cloud Samples](https://cloud.google.com/docs/samples?l=go).
To browse ready to use code samples check [Places API (New) client library examples](https://developers.google.com/maps/documentation/places/web-service/client-library-examples#go).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The updated link is specific to the Places API (New), but this README covers the entire maps module, which includes other APIs like Route Optimization and Solar. Replacing the general samples search link with a specific one makes it harder for users of these other APIs to find relevant samples. Consider keeping the general link alongside the specific one to ensure users of all APIs in this module can find relevant resources.

Suggested change
To browse ready to use code samples check [Places API (New) client library examples](https://developers.google.com/maps/documentation/places/web-service/client-library-examples#go).
To browse ready to use code samples check [Google Cloud Samples](https://cloud.google.com/docs/samples?l=go) or the [Places API (New) client library examples](https://developers.google.com/maps/documentation/places/web-service/client-library-examples#go).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

THis is intentional

@bhshkh bhshkh merged commit 7988942 into googleapis:main May 22, 2026
15 checks passed
@bhshkh bhshkh deleted the chore/maps-samples branch May 22, 2026 18:57
bhshkh added a commit to bhshkh/google-cloud-go that referenced this pull request May 22, 2026
bhshkh pushed a commit that referenced this pull request May 28, 2026
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.15.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/librarian-go@sha256:b04b076f5eedbb5546bd6fc1404969dd3698c8b19c0f34ae815a84ae735a606a
<details><summary>dataplex: v1.35.0</summary>

##
[v1.35.0](dataplex/v1.34.0...dataplex/v1.35.0)
(2026-05-28)

### Features

* update API sources and regenerate (#14661)
([d0cd917](d0cd917c))

</details>


<details><summary>geminidataanalytics: v1.2.0</summary>

##
[v1.2.0](geminidataanalytics/v1.1.0...geminidataanalytics/v1.2.0)
(2026-05-28)

### Features

* update API sources and regenerate (#14661)
([d0cd917](d0cd917c))

</details>


<details><summary>maps: v1.36.0</summary>

##
[v1.36.0](maps/v1.35.0...maps/v1.36.0)
(2026-05-28)

### Features

* update API sources and regenerate (#14661)
([d0cd917](d0cd917c))

### Documentation

* correct the samples (#14632)
([7988942](7988942c))

</details>


<details><summary>saasplatform: v0.8.0</summary>

##
[v0.8.0](saasplatform/v0.7.0...saasplatform/v0.8.0)
(2026-05-28)

### Features

* update API sources and regenerate (#14661)
([d0cd917](d0cd917c))

</details>


<details><summary>security: v1.25.0</summary>

##
[v1.25.0](security/v1.24.0...security/v1.25.0)
(2026-05-28)

### Features

* update API sources and regenerate (#14661)
([d0cd917](d0cd917c))

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants