Skip to content

docs: document PackManifestOptions to make PackManifest reproducible#749

Merged
Wwwsylvia merged 4 commits into
oras-project:mainfrom
wangxiaoxuan273:doc-reproducible
May 20, 2024
Merged

docs: document PackManifestOptions to make PackManifest reproducible#749
Wwwsylvia merged 4 commits into
oras-project:mainfrom
wangxiaoxuan273:doc-reproducible

Conversation

@wangxiaoxuan273

Copy link
Copy Markdown
Contributor

Resolves #748

@codecov

codecov Bot commented Apr 23, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.92%. Comparing base (c071bed) to head (961075d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #749   +/-   ##
=======================================
  Coverage   77.92%   77.92%           
=======================================
  Files          63       63           
  Lines        4694     4694           
=======================================
  Hits         3658     3658           
  Misses        656      656           
  Partials      380      380           

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

@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.

We also need to rephrase the related comment a bit for ExamplePackManifest_imageV10 and ExamplePackManifest_imageV11 in example_pack_test.go.

Comment thread pack.go Outdated
Comment thread pack.go Outdated
Comment thread pack.go
@Wwwsylvia

Copy link
Copy Markdown
Member

BTW, the PR title should start with docs (end with s) as per the conventional commit guidance. 😀

@shizhMSFT shizhMSFT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You also need to update the examples in example_pack_test.go.

Comment thread pack.go Outdated
Comment thread pack.go Outdated
@wangxiaoxuan273 wangxiaoxuan273 changed the title doc: document PackManifestOptions to make PackManifest reproducible docs: document PackManifestOptions to make PackManifest reproducible May 6, 2024
@wangxiaoxuan273

Copy link
Copy Markdown
Contributor Author

We also need to rephrase the related comment a bit for ExamplePackManifest_imageV10 and ExamplePackManifest_imageV11 in example_pack_test.go.

updated the comments in example_pack_test.go.

Comment thread example_pack_test.go Outdated
Comment thread pack.go Outdated
Comment thread pack.go Outdated
Comment thread pack.go

@shizhMSFT shizhMSFT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@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

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>

@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

@Wwwsylvia Wwwsylvia merged commit aef90e4 into oras-project:main May 20, 2024
@wangxiaoxuan273 wangxiaoxuan273 deleted the doc-reproducible branch February 5, 2025 07:11
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.

docs: document PackManifestOptions to make PackManifest reproducible

3 participants