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

chore: remove cosign policy command #2846

Merged
merged 1 commit into from
Apr 6, 2023

Conversation

rdimitrov
Copy link
Contributor

Summary

The following PR removes the cosign policy command.

It is motivated by the discussions in sigstore/sigstore#737 and it is a necessary step in order for sigstore/sigstore#737 to be merged.

Closes #2836

Release Note

Deprecated: Removed cosign policy command

Documentation

Signed-off-by: Radoslav Dimitrov <dimitrovr@vmware.com>
@codecov
Copy link

codecov bot commented Mar 28, 2023

Codecov Report

Merging #2846 (137b40a) into main (b41160e) will increase coverage by 0.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2846      +/-   ##
==========================================
+ Coverage   29.47%   30.32%   +0.84%     
==========================================
  Files         151      149       -2     
  Lines        9678     9399     -279     
==========================================
- Hits         2853     2850       -3     
+ Misses       6386     6110     -276     
  Partials      439      439              
Impacted Files Coverage Δ
cmd/cosign/cli/commands.go 0.00% <ø> (ø)

... and 1 file with indirect coverage changes

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

@hectorj2f hectorj2f left a comment

Choose a reason for hiding this comment

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

lgtm. Shouldn't we add a deprecation message prior its removal for the next release?

@znewman01
Copy link
Contributor

lgtm. Shouldn't we add a deprecation message prior its removal for the next release?

normally, yes, but @asraa claims that it doesn't really work right now

@znewman01
Copy link
Contributor

@rdimitrov can you add a note to that effect in the release notes? otherwise LGTM

@rdimitrov
Copy link
Contributor Author

rdimitrov commented Mar 31, 2023

@rdimitrov can you add a note to that effect in the release notes? otherwise LGTM

Sure, what would be the name of the next release as there's no template for it yet? If of course, I'm looking at the right place? 😃

@rdimitrov rdimitrov closed this Mar 31, 2023
@rdimitrov rdimitrov reopened this Mar 31, 2023
@cpanato cpanato requested review from hectorj2f and asraa April 6, 2023 11:05
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.

Remove cosign policy command
5 participants