CI: Ignore unpinned-uses to github cleanup actions

[bewakes]

Description

zizmor CI checks are failing because the cleanup action does not have a version/tag/branch pinned to it. This PR tags the action with @main.

EDIT: That didn't work so as @Rajil1213 suggested, I've just commented the cleanup actions with # zizmor: ignore[unpinned-uses].

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactor
• New or updated tests
• Dependency Update

Notes to Reviewers

Checklist

• I have performed a self-review of my code.
• I have commented my code where necessary.
• I have updated the documentation if needed.
• My changes do not introduce new warnings.
• I have added tests that prove my changes are effective or that my feature works.
• New and existing tests pass with my changes.

Related Issues

[github-actions]

Commit: c651589

SP1 Performance Test Results

program	cycles	success
BTC_BLOCKSPACE	7,239,474	✅
EL_BLOCK	100,495	✅
CL_BLOCK	55,340	✅
L1_BATCH	12,460,065	✅
L2_BATCH	5,448	✅
CHECKPOINT	15,322	✅

[Rajil1213]

I think the easiest solution is to just add # zizmor: ignore[unpinned-uses] against each line where the cleanup action is used. zizmor will soon support full auditing for composite actions via woodruffw/zizmor#350. And this audit is just a warning. So, it should be fine either way.

We could also open an issue/PR upstream though.

[bewakes]

As @Rajil1213 suggested, I've just commented the cleanup actions with # zizmor: ignore[unpinned-uses]. I haven't done anything about the upstream Issue/PR though.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.77%. Comparing base (cb020cb) to head (b461192).
Report is 2 commits behind head on main.

@@            Coverage Diff             @@
##             main     #608      +/-   ##
==========================================
- Coverage   56.82%   56.77%   -0.06%     
==========================================
  Files         308      308              
  Lines       32383    32397      +14     
==========================================
- Hits        18401    18392       -9     
- Misses      13982    14005      +23     

see 3 files with indirect coverage changes

[Rajil1213]

On a slightly unrelated note, looks like uvx has been deprecated. We should consider moving to uvenv as recommended.

[Rajil1213]

CI changes look fine. So, approving this preemptively but we should wait for a review from Aaron too.

[storopoli]

On a slightly unrelated note, looks like uvx has been deprecated. We should consider moving to uvenv as recommended.

Can I ask you to open a ticket with the context that you have in your working memory so that we don't lose it?

[AaronFeickert]

I think overall this is fine, given it only applies to the cleanup script.