Skip to content

ci(semgrep): consume the fleet-shared reusable Semgrep workflow#1760

Open
krisarmstrong wants to merge 2 commits into
mainfrom
ci/reusable-semgrep
Open

ci(semgrep): consume the fleet-shared reusable Semgrep workflow#1760
krisarmstrong wants to merge 2 commits into
mainfrom
ci/reusable-semgrep

Conversation

@krisarmstrong

Copy link
Copy Markdown
Collaborator

Summary

Replace seed's byte-identical Semgrep job with a call to MustardSeedNetworks/.github's reusable semgrep.yml (Phase 3). ci-complete still needs: [semgrep]; the caller job keeps id semgrep and its result propagates from the reusable workflow.

Verified end-to-end on the niac canary (#1008): reusable Semgrep SAST green + CI Complete green.

Linked Issue

Related to #1751

Testing Evidence

python3 -c "import yaml; yaml.safe_load(open('.github/workflows/ci.yml'))"   # valid

niac #1008 (same reusable workflow): Semgrep SAST pass, CI Complete pass.

Security and Release Checklist

  • Same pinned semgrep==1.168.0 + config/severity; only definition location moved.
  • Aggregated by ci-complete (required check), not itself required — no branch-protection impact.
  • First-party org reusable workflow @main (deliberate, one-edit-point).

Replace seed's byte-identical Semgrep job with a call to
MustardSeedNetworks/.github's reusable semgrep.yml (Phase 3). ci-complete
still needs: [semgrep]. Verified via niac canary (#1008, CI Complete green).
@github-actions github-actions Bot added the area: ci Ci area. label Jul 10, 2026
@krisarmstrong krisarmstrong enabled auto-merge (squash) July 10, 2026 16:14
Pin the license-check + semgrep reusable-workflow refs to
MustardSeedNetworks/.github@v1.0.0 (762b756) instead of @main, matching
the fleet's action-pinning discipline and the .github README's stated
pattern. Renovate bumps these github-actions refs like any other pinned
action (same model as the foundation module), so the one-edit-point holds
while a bad shared-workflow push is gated by each repo's CI rather than
landing instantly fleet-wide.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci Ci area.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant