doc(v3): migration guide edits #15950

scotthart · 2026-02-09T23:25:36Z

This change is

gemini-code-assist · 2026-02-09T23:25:49Z

Summary of Changes

Hello @scotthart, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request refines the v3 migration guide documentation by reorganizing content related to build system changes and decommissioned API surfaces. The changes aim to improve clarity and structure, making it easier for users to navigate information about Bazel module integration and specific header file removals.

Highlights

Documentation Restructuring: The 'Bazel Central Registry' section was relocated and encapsulated within a collapsible details block under a new 'Build system changes' heading for better organization.
Dependency Clarification: The description for the crc32c dependency was updated to specify its distribution as part of Abseil.
Decommissioned API Surfaces: A new top-level section 'Decommissioned API Surfaces' was introduced, and the removal of the iam/retry_traits.h file was moved and documented within this new section.

Changelog

doc/v3-migration-guide.md
- Relocated the 'Bazel Central Registry' section to a new 'Build system changes' heading and wrapped it in a collapsible details block.
- Updated the description for the crc32c dependency to note its inclusion in Abseil.
- Introduced a new top-level section titled 'Decommissioned API Surfaces'.
- Moved and documented the removal of the iam/retry_traits.h file under the new 'Decommissioned API Surfaces' section.
- Removed a duplicate entry for the iam/retry_traits.h file removal.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request improves the v3 migration guide by reorganizing sections for better clarity and adding information about decommissioned APIs. The changes are well-structured and make the guide easier to follow. I have one minor suggestion to improve the grammar in one of the updated lines.

gemini-code-assist · 2026-02-09T23:38:18Z

doc/v3-migration-guide.md

 ### Relocated Dependencies

- `crc32c`
+- `crc32c` Is now distributed as part of Abseil.


For better grammar, the sentence should start with a lowercase letter as it is part of a bulleted list item.

Suggested change

- `crc32c` Is now distributed as part of Abseil.

- `crc32c` is now distributed as part of Abseil.

codecov · 2026-02-09T23:52:09Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.62%. Comparing base (6d0f68d) to head (9778333).
⚠️ Report is 2 commits behind head on prepare-for-v3.0.0.

Additional details and impacted files

@@                 Coverage Diff                 @@
##           prepare-for-v3.0.0   #15950   +/-   ##
===================================================
  Coverage               92.61%   92.62%           
===================================================
  Files                    2332     2332           
  Lines                  213322   213322           
===================================================
+ Hits                   197568   197587   +19     
+ Misses                  15754    15735   -19

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

coryan

A few suggestions in style. I like the use of <details>. The guide is not too long, but provides more information where needed.

"The passive voice should never be used"

coryan · 2026-02-09T23:48:25Z

doc/v3-migration-guide.md

 ### Relocated Dependencies

- `crc32c`
+- `crc32c` Is now distributed as part of Abseil.


I am not sure you need to say that? Maybe just "google-cloud-cpp now uses the crc32c implementation in Abseil"? I am not sure it matters that the implementation is distributed with Abseil or not.

coryan · 2026-02-09T23:49:21Z

doc/v3-migration-guide.md

 ### Bazel

+<details>
+<summary>google-cloud-cpp module in Bazel Central Registry</summary>


Are you still supporting customers with workspaces? If not, just say that?

Updated to describe WORKSPACE support going forward.

coryan · 2026-02-09T23:50:07Z

doc/v3-migration-guide.md

s/the CMakeLists.txt/your CMakeLists.txt`

coryan · 2026-02-09T23:56:47Z

doc/v3-migration-guide.md

s/The deprecated /The deprecated class / ?

PS: you know I do not like the passive voice. "We removed the bigtable::ClientOptions class in this release. We had deprecated this class in ...".

coryan · 2026-02-10T00:08:37Z

doc/v3-migration-guide.md

As opposed to the counterfeit migration guide? Just say "migration guide".

scotthart · 2026-02-10T16:48:08Z

I made a lot of edits to switch from passive voice to active voice.

coryan · 2026-02-10T16:57:18Z

doc/v3-migration-guide.md

 new API surfaces that supersede the previous deprecated types and functions. As
 part of the v3 release series, we are decommissioning those deprecated API
-surfaces. This guide serves a central location to document how to migrate from
+surfaces. This guide provides a central location to document how to migrate from


Are we repeating things?

scotthart requested a review from a team as a code owner February 9, 2026 23:25

scotthart temporarily deployed to false February 9, 2026 23:25 — with GitHub Actions Inactive

scotthart requested a review from coryan February 9, 2026 23:25

gemini-code-assist bot reviewed Feb 9, 2026

View reviewed changes

coryan reviewed Feb 10, 2026

View reviewed changes

scotthart temporarily deployed to false February 10, 2026 16:46 — with GitHub Actions Inactive

coryan approved these changes Feb 10, 2026

View reviewed changes

scotthart temporarily deployed to false February 10, 2026 19:56 — with GitHub Actions Inactive

scotthart enabled auto-merge (squash) February 10, 2026 19:58

scotthart added 4 commits February 10, 2026 15:05

doc(v3): migration guide edits

bcfc2a0

address review comments

1b1ffc4

edits to change from passive voice to active

fbeb6a7

remove repetition

9778333

scotthart force-pushed the v3_clean_migration_guide branch from eb70ac5 to 9778333 Compare February 10, 2026 20:06

scotthart temporarily deployed to false February 10, 2026 20:06 — with GitHub Actions Inactive

scotthart merged commit 2d1b10a into googleapis:prepare-for-v3.0.0 Feb 10, 2026
57 of 64 checks passed

	- `crc32c` Is now distributed as part of Abseil.
	- `crc32c` is now distributed as part of Abseil.

doc(v3): migration guide edits #15950

doc(v3): migration guide edits #15950

Conversation

scotthart commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist bot commented Feb 9, 2026

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

codecov bot commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

coryan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

scotthart commented Feb 10, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

scotthart commented Feb 9, 2026 •

edited

Loading

codecov bot commented Feb 9, 2026 •

edited

Loading