Skip to content

Conversation

@BenWhitehead
Copy link
Collaborator

All Convertable types share a common set of properties that they need to provide. This new class will ensure in a central location they do.

First property: a round trip through the codec should provide a value equal to the initial input

All Convertable types share a common set of properties that they need to provide. This new class will ensure in a central location they do.

First property: a round trip through the codec should provide a value equal to the initial input
@BenWhitehead BenWhitehead requested a review from a team as a code owner May 16, 2022 22:52
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/java-storage API. labels May 16, 2022
Copy link
Contributor

@sydney-munro sydney-munro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BenWhitehead BenWhitehead merged commit 0e923a5 into feat/grpc-storage May 17, 2022
@BenWhitehead BenWhitehead deleted the base-prop-test branch May 17, 2022 22:09
sydney-munro pushed a commit that referenced this pull request Jun 7, 2022
All Convertable types share a common set of properties that they need to provide. This new class will ensure in a central location they do.

First property: a round trip through the codec should provide a value equal to the initial input
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants