Skip to content

chore: disable release for google-crc32c#16521

Merged
parthea merged 1 commit intomainfrom
block-release-for-python-packages
Apr 2, 2026
Merged

chore: disable release for google-crc32c#16521
parthea merged 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
@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 2, 2026
@parthea parthea changed the title chore: disable generation for google-crc32c chore: disable release for google-crc32c Apr 2, 2026
@parthea parthea merged commit 6965854 into main Apr 2, 2026
31 checks passed
@parthea parthea deleted the block-release-for-python-packages branch April 2, 2026 14:30
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