Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

update dependencies, fix clippy warnings, bump to 0.11.0 #276

Merged
merged 4 commits into from May 27, 2022

Conversation

stevelr
Copy link
Contributor

@stevelr stevelr commented May 25, 2022

Signed-off-by: stevelr steve@cosmonic.com

Signed-off-by: stevelr <steve@cosmonic.com>
autodidaddict
autodidaddict previously approved these changes May 26, 2022
Copy link
Member

@brooksmtownsend brooksmtownsend left a comment

Choose a reason for hiding this comment

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

LGTM just needs a little Cargo.toml merging

@thomastaylor312
Copy link
Contributor

thomastaylor312 commented May 26, 2022

Unless we are in a rush to finalize 0.11, I'd recommend we wait until the bugfix release of oci-distribution is released: krustlet/oci-distribution#37. This would make it so we don't have to cut another release

@brooksmtownsend
Copy link
Member

brooksmtownsend commented May 26, 2022

Unless we are in a rush to finalize 0.11, I'd recommend we wait until the bugfix release of oci-distribution is released: krustlet/oci-distribution#37. This would make it so we don't have to cut another release

@thomastaylor312 BTW, this PR #269 already resolved what that bugfix ends up fixing, it'll be a slight internal refactor on our side but won't require a semver bump (edit: except for configurable max chunk limits, which if we remove the flag then it'll just be a simple change)

Signed-off-by: stevelr <steve@cosmonic.com>
Signed-off-by: stevelr <steve@cosmonic.com>
Signed-off-by: stevelr <steve@cosmonic.com>
@brooksmtownsend brooksmtownsend merged commit 1c09908 into main May 27, 2022
@brooksmtownsend brooksmtownsend deleted the finalize-0.11 branch May 27, 2022 12:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants