Oras discover with output tree format #297
Conversation
Signed-off-by: Tejaswini Duggaraju <[email protected]>
Signed-off-by: Tejaswini Duggaraju <[email protected]>
mnltejaswini
force-pushed
the
naduggar/tree
branch
from
1e964d1 to
c87e684
Compare
|
@SteveLasker please review |
Signed-off-by: Tejaswini Duggaraju <[email protected]>
mnltejaswini
force-pushed
the
naduggar/tree
branch
from
3c40537 to
0ad5483
Compare
| @@ -17,12 +17,13 @@ import ( | |||
| ocispec "github.com/opencontainers/image-spec/specs-go/v1" | |||
| "github.com/sirupsen/logrus" | |||
| "github.com/spf13/cobra" | |||
| "github.com/xlab/treeprint" | |||
There was a problem hiding this comment.
This is currently under MIT. Is that a concern or can we take dependency on this?
/cc @lachie83 @SteveLasker
There was a problem hiding this comment.
The use of dependencies with permissive licenses is fine in the CNCF. It's copyleft licenses like MPL and GPL that you need to explicitly ask for an exception to be granted. Here is the doc from the CNCF stating that MIT is allowed - https://github.com/cncf/foundation/blob/master/allowed-third-party-license-policy.md#approved-licenses-for-allowlist
There was a problem hiding this comment.
Also - thanks for paying attention to this @sajayantony! It's not easy.
There was a problem hiding this comment.
The treeview experience in the CLI really helps visualize the graph. Thanks @mnltejaswini for the great experience.
|
LGTM. |
* oras discover with tree output format Signed-off-by: Tejaswini Duggaraju <[email protected]> * oras discover with tree output format Signed-off-by: Tejaswini Duggaraju <[email protected]> * Apply artifact type filter only on the root node Signed-off-by: Tejaswini Duggaraju <[email protected]> Signed-off-by: Sajay Antony <[email protected]>
Add support for tree output to
oras discovercommand. This output format traverses all the references and display them in a hierarchy tree structure.Fixes #295