Skip to content

Update tonic and prost dependencies#194

Merged
mxpv merged 1 commit intocontainerd:mainfrom
mxpv:prost
Sep 8, 2023
Merged

Update tonic and prost dependencies#194
mxpv merged 1 commit intocontainerd:mainfrom
mxpv:prost

Conversation

@mxpv
Copy link
Member

@mxpv mxpv commented Sep 7, 2023

Address #187 #192

@mxpv mxpv added T-dependencies Pull requests that update a dependency file C-client Containerd client C-snapshots Remote snapshotter labels Sep 7, 2023
@github-actions github-actions bot added T-CI Changes in project's CI and removed C-client Containerd client C-snapshots Remote snapshotter labels Sep 7, 2023
@mxpv mxpv added C-client Containerd client C-snapshots Remote snapshotter labels Sep 7, 2023
Burning1020
Burning1020 approved these changes Sep 8, 2023
View reviewed changes
Copy link
Member

@Burning1020 Burning1020 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

@mxpv mxpv added this pull request to the merge queue Sep 8, 2023
Merged via the queue into containerd:main with commit 9388ecf Sep 8, 2023
@mxpv mxpv deleted the prost branch September 8, 2023 17:07
@jsturtevant jsturtevant mentioned this pull request Oct 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsturtevant jsturtevant Awaiting requested review from jsturtevant

2 more reviewers

@Burning1020 Burning1020 Burning1020 approved these changes

@dcantah dcantah dcantah approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

C-client Containerd client C-snapshots Remote snapshotter T-CI Changes in project's CI T-dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mxpv @Burning1020 @dcantah