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

🌱 Create a Generic client to download release assets #187

Merged

Conversation

guettli
Copy link
Contributor

@guettli guettli commented Jun 10, 2024

What this PR does / why we need it:

Current code is opinionated towards the github client for downloading the assets, instead we need a more generic approach that can incorporate other plugins (oci-registry) too.

This PR create a generic assetsclient interface. This interface will be implemented for github and for oci-artifacts.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

@guettli guettli force-pushed the tg/create-generic-client-to-download-release-assets-via-oci branch from 02664fc to f9e696e Compare June 10, 2024 09:13
@guettli guettli requested a review from janiskemper June 10, 2024 09:14
@jschoone jschoone linked an issue Jun 14, 2024 that may be closed by this pull request
@janiskemper janiskemper force-pushed the tg/create-generic-client-to-download-release-assets-via-oci branch 2 times, most recently from 8b637cd to 564bc7e Compare June 24, 2024 13:51
create a generic assetsclient interface

Current code is opinionated towards the github client for
downloading the assets, instead we need a more generic approach that can
incorporate other plugins (oci-registry) too.

Signed-off-by: Thomas Guettler <[email protected]>
Signed-off-by: janiskemper <[email protected]>
@janiskemper janiskemper force-pushed the tg/create-generic-client-to-download-release-assets-via-oci branch from 564bc7e to 454b031 Compare June 24, 2024 13:59
@janiskemper janiskemper merged commit d104052 into main Jun 24, 2024
7 checks passed
@janiskemper janiskemper deleted the tg/create-generic-client-to-download-release-assets-via-oci branch June 24, 2024 14:04
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.

Generic assetsclient interface
2 participants