Skip to content

Conversation

@anguillanneuf
Copy link
Contributor

Create PR after #163 is merged and published on Maven Central.

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 5, 2020
@anguillanneuf anguillanneuf added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 5, 2020
@dpcollins-google dpcollins-google marked this pull request as ready for review August 11, 2020 14:18
@dpcollins-google dpcollins-google requested review from a team and dpcollins-google August 11, 2020 14:18
@dpcollins-google
Copy link
Collaborator

This all LGTM. Going to get the auto-assignment stuff on maven central today, then this should be submittable. If I could figure out how to approve I would, but it won't seem to let me.

Copy link
Collaborator

@dpcollins-google dpcollins-google left a comment

Choose a reason for hiding this comment

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

Approved! Looks good, thanks for also picking up the storage size changes

@anguillanneuf anguillanneuf removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 11, 2020
@anguillanneuf anguillanneuf merged commit 945f775 into master Aug 11, 2020
@anguillanneuf anguillanneuf deleted the samples-subscriber branch August 11, 2020 16:04
@release-please release-please bot mentioned this pull request Aug 12, 2020
@release-please release-please bot mentioned this pull request Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants