Skip to content
This repository was archived by the owner on Mar 3, 2026. It is now read-only.

feat: Set CRC32C as the default checksum option#2635

Closed
thiyaguk09 wants to merge 3 commits intomainfrom
feat/default-crc32c
Closed

feat: Set CRC32C as the default checksum option#2635
thiyaguk09 wants to merge 3 commits intomainfrom
feat/default-crc32c

Conversation

@thiyaguk09
Copy link
Copy Markdown
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #

@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/nodejs-storage API. labels Aug 18, 2025
@thiyaguk09 thiyaguk09 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 18, 2025
@thiyaguk09 thiyaguk09 marked this pull request as ready for review August 18, 2025 18:30
@thiyaguk09 thiyaguk09 requested review from a team August 18, 2025 18:30
@ddelgrosso1
Copy link
Copy Markdown
Contributor

I would consider this a breaking change so this likely needs to be rolled into the next major release.

@ddelgrosso1 ddelgrosso1 added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 18, 2025
@sofisl sofisl closed this Mar 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/nodejs-storage API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants