Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove reference #164

Merged
merged 2 commits into from
Oct 7, 2024
Merged

remove reference #164

merged 2 commits into from
Oct 7, 2024

Conversation

flavio
Copy link
Contributor

@flavio flavio commented Sep 12, 2024

The Reference struct has been moved into the oci-spec crate.

This PR removes the duplicated files and exports Reference from the oci-spec crate.

Consumers of oci-client won't even need the change, unless they were importing Reference from oci_client::reference::Reference instad of oci_client::Reference.

On top of that, I've also addressed some clippy warnings.

The `Reference` struct has been moved into the oci-spec crate.

This commit removes the duplicated files and exports `Reference` from
the oci-spec crate.

Consumers of `oci-client` won't even need the change, unless they were
importing `Reference` from `oci_client::reference::Reference` instad of
`oci_client::Reference`.

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
Address warnings raised by latest version of clippy

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
@flavio
Copy link
Contributor Author

flavio commented Sep 12, 2024

CC @cgwalters

@Xynnn007
Copy link
Contributor

Hey folks, will a new release be published after this PR gets merged?

@flavio
Copy link
Contributor Author

flavio commented Sep 23, 2024

@thomastaylor312 would you mind giving a quick look at this PR?

@Xynnn007 yes, I think we can do that

Copy link
Contributor

@thomastaylor312 thomastaylor312 left a comment

Choose a reason for hiding this comment

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

All good here! Sorry for the delay, got pulled away into other priority work and haven't had time until now to come back. Excited to see us using oci-spec

@flavio flavio merged commit 3e9c25d into oras-project:main Oct 7, 2024
5 checks passed
@flavio flavio deleted the remove-reference branch October 7, 2024 06:54
@flavio
Copy link
Contributor Author

flavio commented Oct 7, 2024

@thomastaylor312 no worries!

flavio added a commit to flavio/rust-oci-client that referenced this pull request Oct 7, 2024
== What's Changed
* client: add support to set https_proxy/no_proxy for client by @Xynnn007 in oras-project#160
* chore(deps): Update testcontainers requirement from 0.20 to 0.22 by @dependabot in oras-project#161
* Docs.rs badge, update crate name. by @bddap in oras-project#166
* chore(deps): Bump actions/checkout from 4.1.7 to 4.2.0 by @dependabot in oras-project#170
* chore(deps): Update testcontainers requirement from 0.22 to 0.23 by @dependabot in oras-project#169
* chore(deps): Update rstest requirement from 0.22.0 to 0.23.0 by @dependabot in oras-project#168
* Allow doing start-end RANGE requests against blobs by @thundergolfer in oras-project#163
* fix(client): Return the digest header for partial responses by @thomastaylor312 in oras-project#171
* remove reference by @flavio in oras-project#164

== New Contributors
* @bddap made their first contribution in oras-project#166
* @thundergolfer made their first contribution in oras-project#163

**Full Changelog**: oras-project/rust-oci-client@v0.12.1...v0.13.0

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
@flavio flavio mentioned this pull request Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants