Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: rename ErrSkipDesc to SkipNode #643

Merged
merged 4 commits into from
Nov 21, 2023

Conversation

wangxiaoxuan273
Copy link
Contributor

Resolves #634

Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

@ktarplee
Copy link
Contributor

I also think we should add a sentence to the documentation for the PreCopy option that describes what happens when that function returns a SkipNode error.

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.

+1 to @ktarplee's comments

@wangxiaoxuan273
Copy link
Contributor Author

wangxiaoxuan273 commented Nov 20, 2023

I also think we should add a sentence to the documentation for the PreCopy option that describes what happens when that function returns a SkipNode error.

Added a doc comment in the PreCopy field.

@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2023

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (9febd7b) 75.19% compared to head (7363a2c) 75.20%.
Report is 1 commits behind head on main.

Files Patch % Lines
content/oci/oci.go 78.72% 8 Missing and 2 partials ⚠️
copy.go 50.00% 1 Missing ⚠️

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #643   +/-   ##
=======================================
  Coverage   75.19%   75.20%           
=======================================
  Files          59       59           
  Lines        5427     5473   +46     
=======================================
+ Hits         4081     4116   +35     
- Misses        992     1001    +9     
- Partials      354      356    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
Signed-off-by: Xiaoxuan Wang <wangxiaoxuan119@gmail.com>
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 3776676 into oras-project:main Nov 21, 2023
@wangxiaoxuan273 wangxiaoxuan273 deleted the rename-var branch November 21, 2023 08:36
@Wwwsylvia Wwwsylvia mentioned this pull request Jan 26, 2024
4 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename ErrSkipDesc to SkipNode
5 participants