Skip to content

chore(tool/cmd/migrate): remove hardcoded excluded_protos for google/rpc#5828

Merged
zhumin8 merged 1 commit intogoogleapis:mainfrom
zhumin8:rm-hardcoded-exclude-rpc
May 5, 2026
Merged

chore(tool/cmd/migrate): remove hardcoded excluded_protos for google/rpc#5828
zhumin8 merged 1 commit intogoogleapis:mainfrom
zhumin8:rm-hardcoded-exclude-rpc

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented May 5, 2026

Remove hardcoded exclusion for google/rpc to keep in sync with recent changes in google-cloud-java. See googleapis/google-cloud-java#12992 and googleapis/google-cloud-java#13005

Fix #5803

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 removes the specific override for the "google/rpc" path in the Java proto migration logic, which previously excluded "google/rpc/http.proto". The corresponding test case in java_test.go has also been removed. I have no feedback to provide as there were no review comments to evaluate.

@zhumin8 zhumin8 marked this pull request as ready for review May 5, 2026 18:37
@zhumin8 zhumin8 requested a review from a team as a code owner May 5, 2026 18:37
@zhumin8 zhumin8 enabled auto-merge (squash) May 5, 2026 18:37
@zhumin8 zhumin8 merged commit a806b1c into googleapis:main May 5, 2026
25 of 26 checks passed
sofisl pushed a commit that referenced this pull request May 5, 2026
…rpc (#5828)

Remove hardcoded exclusion for `google/rpc` to keep in sync with recent
changes in google-cloud-java. See
googleapis/google-cloud-java#12992 and
googleapis/google-cloud-java#13005

Fix #5803
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.

java: remove hardcoded exclude proto google/rpc/http.proto

2 participants