Skip to content

ci: checkout the repo before detecting workflow changes in security-scan - #58

Merged
sbldevnet merged 1 commit into
mainfrom
agent/go-dev/c5d1a6eb3459
Sep 28, 2026
Merged

sbldevnet merged 1 commit into
mainfrom
agent/go-dev/c5d1a6eb3459

Conversation

@sbldevnet

Copy link
Copy Markdown
Owner

Summary

  • security-scan.yml's changes job runs dorny/paths-filter without checking out the repo, so on push events (which need local git history) it fails with git rev-parse --abbrev-ref HEAD exit code 128, cascading to zizmor (which needs: changes).
  • Add an actions/checkout step (pinned by SHA, matching the rest of the repo's workflows) with fetch-depth: 2 before the Detect workflow changes step.

Test plan

  • changes job succeeds on pull_request events
  • changes job succeeds on push events to main
  • A commit that does not touch .github/workflows/** yields workflows=false
  • A commit that touches a workflow file yields workflows=true and zizmor runs

@sbldevnet
sbldevnet merged commit dc39061 into main Sep 28, 2026
5 checks passed
@sbldevnet
sbldevnet deleted the agent/go-dev/c5d1a6eb3459 branch September 28, 2026 10:18
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.

1 participant