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

Remove policy-controller now that it lives in sigstore/policy-controller #1976

Merged
merged 4 commits into from
Jun 8, 2022

Conversation

vaikas
Copy link
Contributor

@vaikas vaikas commented Jun 8, 2022

Signed-off-by: Ville Aikas [email protected]

Summary

Remove policy-controller bits now that they live in sigstore/policy-controller. Large part of the motivation to remove this asap is to make sure that folks do not make changes here that then are not reflected in the sigstore/policy-controller.

Ticket Link

Fixes

Release Note

policy-controller has been moved to sigstore/policy-controller repo.

@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2022

Codecov Report

Merging #1976 (248c129) into main (2ccdb3e) will decrease coverage by 5.96%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1976      +/-   ##
==========================================
- Coverage   34.66%   28.70%   -5.97%     
==========================================
  Files         152      132      -20     
  Lines       10090     8090    -2000     
==========================================
- Hits         3498     2322    -1176     
+ Misses       6222     5461     -761     
+ Partials      370      307      -63     
Impacted Files Coverage Δ
pkg/cosign/rekor_factory.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ccdb3e...248c129. Read the comment docs.

@vaikas vaikas changed the title [WIP] Remove policy-controller now that it lives in sigstore/policy-controller Remove policy-controller now that it lives in sigstore/policy-controller Jun 8, 2022
@dlorenc
Copy link
Member

dlorenc commented Jun 8, 2022

DELETE!

@dlorenc dlorenc merged commit aa700bf into sigstore:main Jun 8, 2022
@github-actions github-actions bot added this to the v1.10.0 milestone Jun 8, 2022
@vaikas vaikas deleted the remove-policy-controller branch June 9, 2022 05:31
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