Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Nov 20, 2023

This PR is based on the spec PR: #809

It adds the notation plugin uninstall command.

Tests: The e2e test cases are also included in this PR.

Note: This PR depends on the notation-go PR: notaryproject/notation-go#363

Update: PR notaryproject/notation-go#363 has been merged.

Patrick Zheng added 18 commits October 19, 2023 16:50
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
fix
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost changed the title feat: added notation plugin uninstall command feat: notation plugin uninstall command Nov 20, 2023
Patrick Zheng added 2 commits November 20, 2023 13:01
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2023

Codecov Report

Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (acb9707) 64.05% compared to head (2741c85) 64.39%.

Files Patch % Lines
cmd/notation/plugin/uninstall.go 69.35% 13 Missing and 6 partials ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #832      +/-   ##
==========================================
+ Coverage   64.05%   64.39%   +0.33%     
==========================================
  Files          40       42       +2     
  Lines        2323     2390      +67     
==========================================
+ Hits         1488     1539      +51     
- Misses        709      720      +11     
- Partials      126      131       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Patrick Zheng added 5 commits November 21, 2023 08:04
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost requested review from priteshbandi and shizhMSFT November 21, 2023 02:11
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost requested a review from shizhMSFT November 23, 2023 06:15
Patrick Zheng added 3 commits November 24, 2023 18:15
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
Signed-off-by: Patrick Zheng <patrickzheng@microsoft.com>
@ghost ghost closed this Nov 28, 2023
@ghost ghost deleted the plugin-uninstall branch November 28, 2023 09:02
This pull request was closed.
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.

4 participants