Skip to content

Directory Structure Implementation#265

Merged
shizhMSFT merged 1 commit intonotaryproject:mainfrom
JeyJeyGao:feat/directory_struct
Aug 5, 2022
Merged

Directory Structure Implementation#265
shizhMSFT merged 1 commit intonotaryproject:mainfrom
JeyJeyGao:feat/directory_struct

Conversation

@JeyJeyGao
Copy link
Copy Markdown
Contributor

@JeyJeyGao JeyJeyGao commented Jul 26, 2022

  1. support dir package
  2. supoort notation-go/config package

Test:
tested with the workflow mentioned in hello-signing

  1. notation cert generate-test --default "wabbit-networks.io"
  2. notation --plain-http sign $IMAGE
  3. notation --plain-http list $IMAGE
  4. notation --plain-http verify $IMAGE
  5. notation cache list
  6. notation plugin list
  7. notation pull $IMAGE
  8. notation push $IMAGE
  9. notation key list
  10. notation cert add --name "wabbit-networks.io" ~/.config/notation/localkeys/wabbit-networks.io.crt

Signed-off-by: Junjie Gao 43160897+JeyJeyGao@users.noreply.github.com

@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch from 1f5d78b to d049d65 Compare July 26, 2022 07:59
@JeyJeyGao JeyJeyGao changed the title Directory Structure Implementation Directory Structure Implementation (DO NOT MERGED) Jul 26, 2022
@JeyJeyGao JeyJeyGao changed the title Directory Structure Implementation (DO NOT MERGED) Directory Structure Implementation (DO NOT MERGE) Jul 27, 2022
@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch from d049d65 to c09b0c4 Compare July 27, 2022 03:09
@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch 3 times, most recently from 134566e to 464b26d Compare July 29, 2022 06:13
@shizhMSFT shizhMSFT linked an issue Aug 2, 2022 that may be closed by this pull request
5 tasks
@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch 3 times, most recently from 6cafdab to 7a3ba80 Compare August 2, 2022 05:35
@JeyJeyGao JeyJeyGao changed the title Directory Structure Implementation (DO NOT MERGE) Directory Structure Implementation Aug 2, 2022
@dtzar dtzar requested a review from a team August 3, 2022 00:13
@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch from 7a3ba80 to e72f022 Compare August 3, 2022 01:29
Copy link
Copy Markdown
Contributor

@binbin-li binbin-li left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@chloeyin chloeyin left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@patrickzheng200 patrickzheng200 left a comment

Choose a reason for hiding this comment

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

lgtm

@dtzar dtzar added this to the alpha-3 milestone Aug 3, 2022
@dtzar
Copy link
Copy Markdown

dtzar commented Aug 3, 2022

Per the PR review criteria @notaryproject/notation-sub-group-maintainers - this is past a week old and has two approvals from official code-reviewers, so it is eligible for merge now.

@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch from e72f022 to af09540 Compare August 4, 2022 11:03
1. support dir package
2. supoort notation-go/config package

Signed-off-by: Junjie Gao <43160897+JeyJeyGao@users.noreply.github.com>
@JeyJeyGao JeyJeyGao force-pushed the feat/directory_struct branch from af09540 to a181c2c Compare August 5, 2022 01:17
Copy link
Copy Markdown
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

@JeyJeyGao JeyJeyGao deleted the feat/directory_struct branch December 1, 2022 05:58
7h3-3mp7y-m4n pushed a commit to 7h3-3mp7y-m4n/notation that referenced this pull request Mar 29, 2025
1. support dir package
2. supoort notation-go/config package

Signed-off-by: Junjie Gao <43160897+JeyJeyGao@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[User Story] - notation generates key and corresponding certificate for testing

6 participants