Repository navigation
fix(ci): pin actions/checkout v7 for Super-linter - #89
Conversation
Hash-pin checkout v7.0.1 and the other uses in the same workflow files so zizmor unpinned-uses and YAML_PRETTIER pass. Co-Authored-By: duyetbot <duyetbot@users.noreply.github.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reachedNext included review available in 52 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Welcome! 👋Thank you for your first contribution to this project! We appreciate you taking the time to improve our codebase. Next Steps
Contribution Guidelines
Thanks again for contributing! 🎉 |
Reviewer's GuideUpdates four CI workflows to use immutable, verified action commits—including actions/checkout v7.0.1—with checkout credentials disabled, while addressing Super-linter, ShellCheck, template-injection, and Prettier findings without changing plugin catalogs or unrelated workflows. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Co-Authored-By: duyetbot <duyetbot@users.noreply.github.com>
Move interpolations into env and create releases with gh so Super-linter treats zizmor as clean. Co-Authored-By: duyetbot <duyetbot@users.noreply.github.com>
Keep the SHA-pinned release action. zizmor 1.23.1 accepts the ignore on the step name line. Co-Authored-By: duyetbot <duyetbot@users.noreply.github.com>
Why
Renovate opened #79 to bump
actions/checkoutfromv6to thev7tag. Super-linter failed that PR. zizmorunpinned-usesrejects tag refs under the blanket pin policy, and YAML_PRETTIER failed on three of the four files. Master Lint stays green because Super-linter only checks changed files, so the same unpinned uses on master are not in the lint set until a workflow file changes.This PR starts from current
masterand pins the verifiedv7.0.1commit instead of the moving tag. Close #79 as superseded after this lands.Scope
Touched files:
.github/workflows/bash-tests.yml.github/workflows/pr-automation.yml.github/workflows/release.ymlclickhouse/.github/workflows/sync-rules.ymlCheckout uses the SHA
3d3c42e5aac5ba805825da76410c181273ba90b1(v7andv7.0.1both resolve to it) pluspersist-credentials: false. Comments name the human version, same pattern as.github/workflows/validate-plugins.yml.Super-linter also audits every
uses:in a changed file. These pins are required for green Lint on the same files:ludeeus/action-shellcheck@2.0.0→00cae500b08a931fb5698e11e79bfbd38e612a38bats-core/bats-action@4.0.0→77d6fb60505b4d0d1d73e48bd035b55074bbfb43softprops/action-gh-release@v3(v3.0.3) →efb35369e0ad2afab669f228072c1b0d510eae64with# zizmor: ignore[superfluous-actions]on the step name (verified against zizmor 1.23.1)peter-evans/create-pull-request@v8(v8.1.1) →5f6978faf089d4d20b00c7766989d076bb2fc7f1Lint-only fixes in those files:
$GITHUB_OUTPUT(actionlint / ShellCheck SC2086)env(template-injection)printffor the release-notes heading (SC2028)Out of scope: other workflows still on checkout
v4.2.2SHAs (lint.yml,validate-plugins.yml,sync-skills.yml).Blast Radius
CI workflows only. Jobs that check out the repo now run checkout v7.0.1 and do not persist credentials after checkout.
peter-evans/create-pull-requeststill receivestokenin its own inputs. Plugin catalogs and manifests are unchanged.Verification
actions/checkout@v7and@v7.0.1both point at3d3c42e5aac5ba805825da76410c181273ba90b1npx prettier@3.8.1 --checkon the four files: passzizmor1.23.1 on the four files: no findings (1 ignored)node .cursor/skills/verify-marketplace/control-marketplace.mjs doctor --json:oknode .cursor/skills/verify-marketplace/control-marketplace.mjs info --json:ok, catalogs unchanged so the skill was not refreshedSummary by Sourcery
Harden the affected CI workflows by pinning action dependencies and resolving shell and YAML lint issues.
Bug Fixes:
Enhancements:
Chores: