Skip to content

feat: support deletion for memory resolver#607

Merged
TerryHowe merged 3 commits into
oras-project:mainfrom
wangxiaoxuan273:resolver
Oct 2, 2023
Merged

feat: support deletion for memory resolver#607
TerryHowe merged 3 commits into
oras-project:mainfrom
wangxiaoxuan273:resolver

Conversation

@wangxiaoxuan273

Copy link
Copy Markdown
Contributor

Part 2/4 of #454

Based on draft PR #582

@codecov-commenter

codecov-commenter commented Sep 25, 2023

Copy link
Copy Markdown

Codecov Report

Merging #607 (50caeb4) into main (2d371a0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##             main     #607   +/-   ##
=======================================
  Coverage   74.46%   74.47%           
=======================================
  Files          59       59           
  Lines        5275     5277    +2     
=======================================
+ Hits         3928     3930    +2     
  Misses        993      993           
  Partials      354      354           
Files Coverage Δ
internal/resolver/memory.go 100.00% <100.00%> (ø)

Comment thread internal/resolver/memory.go Outdated
Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>

@Wwwsylvia Wwwsylvia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@TerryHowe TerryHowe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@TerryHowe TerryHowe merged commit 6456d16 into oras-project:main Oct 2, 2023
@wangxiaoxuan273 wangxiaoxuan273 deleted the resolver branch October 7, 2023 03:34
shizhMSFT pushed a commit that referenced this pull request Nov 20, 2023
Part 4/4 of #454 

This PR should be reviewed and merged after PR #606 #607 #608

---------

Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
@Wwwsylvia Wwwsylvia mentioned this pull request Jan 26, 2024
4 tasks
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.

5 participants