Skip to content

Conversation

TerryHowe
Copy link
Member

What this PR does / why we need it:

Every time I'm in this file, I get confused about what opts is because that is used outside this everywhere for options

Copy link
Member

@Wwwsylvia Wwwsylvia left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@wangxiaoxuan273 wangxiaoxuan273 left a comment

Choose a reason for hiding this comment

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

LGTM

@Wwwsylvia
Copy link
Member

Oops there are some merge conflicts @TerryHowe

Signed-off-by: Terry Howe <terrylhowe@gmail.com>
@TerryHowe TerryHowe force-pushed the fix-target-receiver branch from 99c33a1 to b7a1495 Compare March 27, 2025 15:32
@TerryHowe
Copy link
Member Author

Conflicts resolved

Copy link
Member

@Wwwsylvia Wwwsylvia left a comment

Choose a reason for hiding this comment

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

LGTM

@Wwwsylvia Wwwsylvia merged commit 6f04cf0 into oras-project:main Mar 28, 2025
6 checks passed
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.

3 participants