Skip to content

Revert "fix(generator): use flat_ref_types in test templates and delete remove-unused-imports" - #18088

Merged
ohmayr merged 1 commit into
mainfrom
revert-17900-delete-remove-unused-imports-script
Aug 12, 2026
Merged

Revert "fix(generator): use flat_ref_types in test templates and delete remove-unused-imports"#18088
ohmayr merged 1 commit into
mainfrom
revert-17900-delete-remove-unused-imports-script

Conversation

@ohmayr

@ohmayr ohmayr commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Reverts #17900

@ohmayr
ohmayr requested a review from a team as a code owner August 12, 2026 16:36

@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 updates the GAPIC generator templates to use method.ref_types instead of method.flat_ref_types when generating unit tests. This change ensures that all referenced types are correctly imported, as reflected in the updated golden test files which now include additional protobuf and Google API imports (such as duration_pb2, http_request_pb2, and struct_pb2). There are no review comments, and I have no feedback to provide.

@ohmayr
ohmayr merged commit a9246fa into main Aug 12, 2026
163 of 177 checks passed
@ohmayr
ohmayr deleted the revert-17900-delete-remove-unused-imports-script branch August 12, 2026 17:42
hebaalazzeh pushed a commit that referenced this pull request Aug 13, 2026
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