Skip to content

chore: complete migration of googleapis-common-protos to librarian#16688

Merged
parthea merged 3 commits intogoogleapis:mainfrom
jskeet:regenerate-googleapis-common-protos
Apr 16, 2026
Merged

chore: complete migration of googleapis-common-protos to librarian#16688
parthea merged 3 commits intogoogleapis:mainfrom
jskeet:regenerate-googleapis-common-protos

Conversation

@jskeet
Copy link
Copy Markdown
Contributor

@jskeet jskeet commented Apr 16, 2026

  • Adds a string replacement to avoid double copyright notices while we're (still) not generating google/gapic and google/longrunning
  • Removes the skip_generate from librarian.yaml
  • Reformats with nox
  • Regenerates

Fixes googleapis/librarian#5334

jskeet added 3 commits April 16, 2026 12:46
Stop skipping generation, and add a new post-processing replacement so
that existing files which aren't currently being regenerated don't end
up with extra copyright statements.
Command: `nox -s format --no-install --no-venv`
@jskeet jskeet requested a review from parthea April 16, 2026 12:51
@jskeet jskeet requested a review from a team as a code owner April 16, 2026 12:51
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 introduces a post-processing configuration to prevent duplicate copyright notices in generated Python files and updates metadata and documentation for the googleapis-common-protos package. Additionally, it reorders imports in several generated protobuf files. A critical issue was identified in the new YAML configuration where a literal backslash-n character in a block scalar may prevent the regex from matching the intended code pattern.

@parthea parthea self-assigned this Apr 16, 2026
@parthea parthea merged commit a81ea85 into googleapis:main Apr 16, 2026
30 checks passed
chalmerlowe pushed a commit that referenced this pull request Apr 17, 2026
…16688)

- Adds a string replacement to avoid double copyright notices while
we're (still) not generating google/gapic and google/longrunning
- Removes the skip_generate from librarian.yaml
- Reformats with nox
- Regenerates

Fixes googleapis/librarian#5334
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.

python: fix and enable generation for googleapis-common-protos

2 participants