Skip to content

feat(dart): export rpc exceptions#7044

Merged
brianquinlan merged 2 commits into
googleapis:mainfrom
brianquinlan:rpc_export
Jul 25, 2026
Merged

feat(dart): export rpc exceptions#7044
brianquinlan merged 2 commits into
googleapis:mainfrom
brianquinlan:rpc_export

Conversation

@brianquinlan

Copy link
Copy Markdown
Contributor

I asked gemini to generate a friction log when using a generated package and having to find where exceptions live was the biggest issue.

I asked gemini to generate a friction log when using a generated package and having to find where exceptions live was the biggest issue.
@brianquinlan
brianquinlan requested a review from a team as a code owner July 25, 2026 00:41

@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 Dart template main.dart.mustache to conditionally export package:google_cloud_rpc/exceptions.dart when Codec.HasServices is true. The reviewer noted that when Codec.HasServices is false, the template leaves multiple consecutive blank lines, violating the repository style guide. They suggested moving the blank line inside the conditional block to prevent this issue.

Comment thread internal/sidekick/dart/templates/lib/main.dart.mustache
@brianquinlan
brianquinlan merged commit 8d98203 into googleapis:main Jul 25, 2026
37 of 38 checks passed
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