Skip to content

chore(librarian): remove post processing script for ces - #18092

Merged
hebaalazzeh merged 1 commit into
mainfrom
remove-ces-post-processing
Aug 12, 2026
Merged

chore(librarian): remove post processing script for ces#18092
hebaalazzeh merged 1 commit into
mainfrom
remove-ces-post-processing

Conversation

@ohmayr

@ohmayr ohmayr commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Removes the temporary post-processing configuration file .librarian/generator-input/client-post-processing/temporary-post-processing.yaml for google-cloud-ces, as the generator fix from #17914 has been released and this workaround is no longer needed.

@ohmayr
ohmayr requested a review from a team as a code owner August 12, 2026 22:31
@hebaalazzeh hebaalazzeh self-assigned this Aug 12, 2026

@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 removes a temporary post-processing configuration file that was used to apply type annotations to header_params in the generated client files for google-cloud-ces. Since the upstream issue has been resolved, this temporary workaround is no longer needed. There are no review comments to address, and we have no additional feedback to provide.

@hebaalazzeh hebaalazzeh 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.

LGTM

@hebaalazzeh
hebaalazzeh enabled auto-merge (squash) August 12, 2026 22:32
@hebaalazzeh
hebaalazzeh merged commit bd521f2 into main Aug 12, 2026
37 checks passed
@hebaalazzeh
hebaalazzeh deleted the remove-ces-post-processing branch August 12, 2026 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants