Skip to content

Remove unused policy evaluation code#4936

Open
aaronlew02 wants to merge 2 commits into
sigstore:mainfrom
aaronlew02:delete-policy
Open

Remove unused policy evaluation code#4936
aaronlew02 wants to merge 2 commits into
sigstore:mainfrom
aaronlew02:delete-policy

Conversation

@aaronlew02

Copy link
Copy Markdown
Contributor

Summary

This change removes unused policy evaluation code obsoleted by #1976:

  • /pkg/policy and associated fuzz tests
  • ValidateJSONWithModuleInput from /pkg/cosign/rego

Signed-off-by: Aaron Lew <64337293+aaronlew02@users.noreply.github.com>
Signed-off-by: Aaron Lew <64337293+aaronlew02@users.noreply.github.com>
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.62%. Comparing base (2ef6022) to head (02059ba).
⚠️ Report is 775 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4936      +/-   ##
==========================================
- Coverage   40.10%   38.62%   -1.48%     
==========================================
  Files         155      220      +65     
  Lines       10044    13524    +3480     
==========================================
+ Hits         4028     5224    +1196     
- Misses       5530     7538    +2008     
- Partials      486      762     +276     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aaronlew02 aaronlew02 marked this pull request as ready for review June 10, 2026 00:57
@aaronlew02 aaronlew02 requested a review from a team as a code owner June 10, 2026 00:57

@steiza steiza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How are you finding these 😅

At any rate, LGTM!

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.

2 participants