-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add async OCI and OCM packages #1053
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
as for the "actual" changes, I have some concerns w.r.t. new package-names. considering other patterns I have seen, it seem to be more common to have (few) different entrypoints (e.g. Specifically for How about a layout like the following (limited to
|
4515562
to
3c3ffd3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm - please rm aiohttp from "hard" dependencies, as suggested by @zkdev
3c3ffd3
to
d82ca63
Compare
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note: