Add Sub-Project clarification #36
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
SteveLasker
requested review from
deitch,
jdolitsky,
sajayantony and
shizhMSFT
as code owners
This was referenced Mar 1, 2023
shizhMSFT
reviewed
Mar 15, 2023
There was a problem hiding this comment.
Do you want to also update the README.md?
There was a problem hiding this comment.
yes, but that could be a separate PR
TerryHowe
reviewed
Mar 15, 2023
Signed-off-by: Steve Lasker <stevenlasker@hotmail.com>
SteveLasker
requested review from
shizhMSFT and
TerryHowe
and removed request for
deitch and
jdolitsky
FeynmanZhou
reviewed
Jul 24, 2023
| | - | - | - | | ||
| | Org Governance | [governance](https://github.com/oras-project/community) | [CODEOWNERS](https://github.com/oras-project/community/blob/main/CODEOWNERS) | | ||
| | [oras.land](https://oras.land) website content | [oras-www](https://github.com/oras-project/oras-www) | [CODEOWNERS](https://github.com/oras-project/oras-www/blob/main/CODEOWNERS) | | ||
| | oras go client | [oras](https://github.com/oras-project/oras)<br>[oras-go](https://github.com/oras-project/oras-go)<br>[setup-oras](https://github.com/oras-project/setup-oras)<br>[oras-credentials-go](https://github.com/oras-project/oras-credentials-go) | [CODEOWNERS](https://github.com/oras-project/oras/blob/main/CODEOWNERS) <br>[CODEOWNERS](https://github.com/oras-project/oras-go/blob/main/CODEOWNERS)<br>[CODEOWNERS](https://github.com/oras-project/setup-oras/blob/main/CODEOWNERS) | |
There was a problem hiding this comment.
@SteveLasker Is it better to differentiate CLI and library for ORAS? It's a bit strange to put oras in the oras go client sub-project. Shall we create a new row ORAS CLI and move oras accordingly?
FeynmanZhou
reviewed
Jul 24, 2023
| @@ -0,0 +1,13 @@ | |||
| # ORAS Sub-Projects | |||
|
|
|||
| The ORAS Project represents the following sub-projects: | |||
There was a problem hiding this comment.
Do we need to clarify the definition differences between sub-project and repository? This doc indicates that the sub-project is one or a group of repositories in the same category.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds
/sub-projects.mdto clarify the list of sub-projects and the associated owners.Fixes #23
FWIW, the line wrapping within the table is not easy to view in markdown editing mode, but viewing in preview mode makes it more clear.
Personally, I prefer the table view for end-user view. But, if folks would prefer a better editing experience, we could switch this to a bulleted list. Although, the mapping between multiple repos and owners would be difficult.