Skip to content
This repository was archived by the owner on Apr 7, 2024. It is now read-only.

docs: add a brief introduction in README#83

Merged
Wwwsylvia merged 7 commits into
oras-project:mainfrom
Wwwsylvia:readme
Jul 14, 2023
Merged

docs: add a brief introduction in README#83
Wwwsylvia merged 7 commits into
oras-project:mainfrom
Wwwsylvia:readme

Conversation

@Wwwsylvia

@Wwwsylvia Wwwsylvia commented Jul 13, 2023

Copy link
Copy Markdown
Member

Resolves: #58

@Wwwsylvia
doc: add a brief introduction in README
280d60c 
Signed-off-by: Lixia (Sylvia) Lei <lixlei@microsoft.com>
@Wwwsylvia
remove empty line
04c047d 
Signed-off-by: Lixia (Sylvia) Lei <lixlei@microsoft.com>
@Wwwsylvia Wwwsylvia changed the title doc: add a brief introduction in README docs: add a brief introduction in README Jul 13, 2023
@codecov-commenter

codecov-commenter commented Jul 13, 2023 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #83 (97df607) into main (ab5d0bd) will increase coverage by 0.18%.
The diff coverage is 85.18%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   82.27%   82.46%   +0.18%     
==========================================
  Files           6        6              
  Lines         378      382       +4     
==========================================
+ Hits          311      315       +4     
  Misses         45       45              
  Partials       22       22
Impacted Files Coverage Δ
native_store.go 83.33% <66.66%> (-3.63%) ⬇️
registry.go 81.25% <85.71%> (-2.09%) ⬇️
store.go 80.95% <100.00%> (+4.12%) ⬆️

TerryHowe
TerryHowe reviewed Jul 13, 2023
View reviewed changes
Comment thread README.md Outdated
@Wwwsylvia
revised
b540d8f 
Signed-off-by: Sylvia Lei <lixlei@microsoft.com>
@Wwwsylvia

Wwwsylvia commented Jul 14, 2023 •
edited
Loading

Copy link
Copy Markdown
Member Author

Codecov Report

Merging #83 (04c047d) into main (ab5d0bd) will increase coverage by 0.18%.
The diff coverage is 85.18%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   82.27%   82.46%   +0.18%     
==========================================
  Files           6        6              
  Lines         378      382       +4     
==========================================
+ Hits          311      315       +4     
  Misses         45       45              
  Partials       22       22

Impacted Files Coverage Δ
native_store.go 83.33% <66.66%> (-3.63%) ⬇️
registry.go 81.25% <85.71%> (-2.09%) ⬇️
store.go 80.95% <100.00%> (+4.12%) ⬆️

Wondering why this PR triggers a test coverage change... 🤨

@Wwwsylvia Wwwsylvia requested a review from TerryHowe July 14, 2023 02:44
FeynmanZhou
FeynmanZhou reviewed Jul 14, 2023
View reviewed changes
Comment thread README.md Outdated
shizhMSFT
shizhMSFT reviewed Jul 14, 2023
View reviewed changes
Comment thread README.md Outdated
Comment thread README.md Outdated
Wwwsylvia added 4 commits July 14, 2023 14:12
@Wwwsylvia
revise
7162a7d 
Signed-off-by: Sylvia Lei <lixlei@microsoft.com>
@Wwwsylvia
fix format
f463566 
Signed-off-by: Sylvia Lei <lixlei@microsoft.com>
@Wwwsylvia
minor format
4c36db2 
Signed-off-by: Sylvia Lei <lixlei@microsoft.com>
@Wwwsylvia
reword
97df607 
Signed-off-by: Sylvia Lei <lixlei@microsoft.com>
shizhMSFT
shizhMSFT approved these changes Jul 14, 2023
View reviewed changes

@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 merged commit 2afb422 into oras-project:main Jul 14, 2023
@Wwwsylvia Wwwsylvia deleted the readme branch July 14, 2023 08:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@TerryHowe TerryHowe Awaiting requested review from TerryHowe TerryHowe is a code owner

2 more reviewers

@FeynmanZhou FeynmanZhou FeynmanZhou left review comments

@shizhMSFT shizhMSFT shizhMSFT approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update README

5 participants

@Wwwsylvia @codecov-commenter @TerryHowe @shizhMSFT @FeynmanZhou