Skip to content

feat(google/cloud/productregistry/v1): add google-cloud-productregistry#17783

Merged
parthea merged 1 commit into
googleapis:mainfrom
suztomo:onboard-new-library-2026-07-20-google/cloud/productregistry/v1
Jul 21, 2026
Merged

feat(google/cloud/productregistry/v1): add google-cloud-productregistry#17783
parthea merged 1 commit into
googleapis:mainfrom
suztomo:onboard-new-library-2026-07-20-google/cloud/productregistry/v1

Conversation

@suztomo

@suztomo suztomo commented Jul 20, 2026

Copy link
Copy Markdown
Member

Introducing google-cloud-productregistry package.

b/530253265

PiperOrigin-RevId: 941236329

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

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.

Code Review

This pull request adds the new google-cloud-productregistry client library package, including its configuration, documentation, generated client code, samples, and tests. It also refactors the import profiler script in scripts/import_profiler/profiler.py to simplify output formatting and statistical calculations. A potential NameError was identified in the profiler script where baseline_p50 could remain undefined if baseline_times is empty.

Comment on lines 247 to +248
if baseline_times:
baseline_p50, _, _ = _calculate_percentiles(baseline_times)
baseline_p50 = statistics.median(baseline_times)

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

If baseline_times is empty (e.g., if the baseline CSV file exists but contains no data rows), baseline_p50 will remain undefined. This will cause a NameError when evaluated on line 250 (if baseline_p50 is not None:).

Using a conditional expression ensures baseline_p50 is always defined (defaulting to None if the list is empty).

            baseline_p50 = statistics.median(baseline_times) if baseline_times else None

@suztomo
suztomo force-pushed the onboard-new-library-2026-07-20-google/cloud/productregistry/v1 branch from e2e797f to b17eacf Compare July 20, 2026 19:05
@suztomo

suztomo commented Jul 20, 2026

Copy link
Copy Markdown
Member Author

I put these pull requests as draft as you may want to setup OSS Exit Gates and PyPI registration before merging this.

@parthea parthea changed the title onboard new library 2026 07 20 google/cloud/productregistry/v1 feat(google/cloud/productregistry/v1): add google-cloud-productregistry Jul 21, 2026
@parthea
parthea marked this pull request as ready for review July 21, 2026 15:18
@parthea
parthea requested a review from a team as a code owner July 21, 2026 15:18
@snippet-bot

snippet-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 14 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

@parthea
parthea merged commit b7fa7df into googleapis:main Jul 21, 2026
39 checks passed
parthea added a commit that referenced this pull request Jul 21, 2026
🤖 I have created a release *beep* *boop*
---


## 0.1.0 (2026-07-21)


### Features

* **google/cloud/productregistry/v1:** add google-cloud-productregistry
([#17783](#17783))
([b7fa7df](b7fa7df))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants