Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

docs: Add development environment section to readme #822

Merged
merged 1 commit into from
Mar 7, 2023

Conversation

TerryHowe
Copy link
Member

Each ORAS sub-project should have a section in the README.md about setting up a development environment, since each sub-project is unique.

Copy link

@deeepsig deeepsig left a comment

Choose a reason for hiding this comment

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

Second this! Thanks for the PR @TerryHowe. I've given some minor suggestions that would help fix some unwanted behavior.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2023

Codecov Report

Merging #822 (da0b027) into main (4d97bbd) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #822   +/-   ##
=======================================
  Coverage   63.50%   63.50%           
=======================================
  Files          21       21           
  Lines         844      844           
=======================================
  Hits          536      536           
  Misses        265      265           
  Partials       43       43           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@SteveLasker SteveLasker left a comment

Choose a reason for hiding this comment

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

Being more explicit on the details helps folks get started.
Thanks for the addition, @TerryHowe

README.md Outdated Show resolved Hide resolved
@deeepsig
Copy link

LGTM 🚀

README.md Outdated Show resolved Hide resolved
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.

We should mention and probably merge this doc https://oras.land/cli/5_developer_guide/?

Copy link
Contributor

@sajayantony sajayantony left a comment

Choose a reason for hiding this comment

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

Oh yes why don't we point to - https://oras.land/cli/5_developer_guide/

And add to that any improvements?
Would avoid things going stale and having to keep them consistent in 2 places.

Copy link
Contributor

@qweeah qweeah left a comment

Choose a reason for hiding this comment

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

LGTM. BTW @TerryHowe, the merging rule now checks for unresolved discussion. If you want to merge this PR, you need to resolve all the comments.

@TerryHowe
Copy link
Member Author

Updated the docs here oras-project/oras-www#119 and just linked to them in this PR. This was part of the CNCF template work to have a development guide in the readme.

Copy link
Member

@FeynmanZhou FeynmanZhou left a comment

Choose a reason for hiding this comment

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

oras-project/oras-www#119 has been approved and merged. I think it's ready to accept this one.

Signed-off-by: Terry Howe <tlhowe@amazon.com>
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

@shizhMSFT shizhMSFT merged commit 03bc2b2 into oras-project:main Mar 7, 2023
@qweeah qweeah mentioned this pull request Mar 17, 2023
5 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.

8 participants