Skip to content

chore: disable generation for google-crc32c#16521

Open
ohmayr wants to merge 1 commit intomainfrom
block-release-for-python-packages
Open

chore: disable generation for google-crc32c#16521
ohmayr wants to merge 1 commit intomainfrom
block-release-for-python-packages

Conversation

@ohmayr
Copy link
Copy Markdown
Contributor

@ohmayr ohmayr commented Apr 1, 2026

Disable generation for google-crc32c.

@ohmayr ohmayr requested review from a team as code owners April 1, 2026 21:33
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 updates the librarian configuration by modifying a tracking issue link and replacing 'google-cloud-dialogflow' with 'google-crc32c' in the blocked generation list. Feedback points out that the descriptive comment was not updated to reflect the new library ID, creating an inconsistency, and questions whether 'google-crc32c' should have been added as a new entry rather than replacing the existing one.

# TODO(https://github.com/googleapis/google-cloud-python/issues/16520):
# Allow generation for google-cloud-dialogflow once this bug is fixed.
- id: "google-cloud-dialogflow"
- id: "google-crc32c"
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.

high

The library google-cloud-dialogflow is being replaced by google-crc32c. If the intention was to block both, google-crc32c should be added as a new entry. Additionally, the comment on line 36 still refers to google-cloud-dialogflow, making it inconsistent with the updated ID and issue link.

@ohmayr ohmayr added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 1, 2026
@ohmayr ohmayr force-pushed the block-release-for-python-packages branch 2 times, most recently from ff3c5ab to 4925532 Compare April 1, 2026 23:24
@ohmayr ohmayr force-pushed the block-release-for-python-packages branch 2 times, most recently from f98b221 to e3b4144 Compare April 2, 2026 07:46
@ohmayr ohmayr force-pushed the block-release-for-python-packages branch from e3b4144 to eb7a6a1 Compare April 2, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants