Skip to content

chore(ci): smoke-test gitleaks workflow post-OPS-732 (do not merge) - #33

Closed
yotamleo wants to merge 1 commit into
masterfrom
chore/ci-smoke-test-gitleaks-pr-target
Closed

chore(ci): smoke-test gitleaks workflow post-OPS-732 (do not merge)#33
yotamleo wants to merge 1 commit into
masterfrom
chore/ci-smoke-test-gitleaks-pr-target

Conversation

@yotamleo

@yotamleo yotamleo commented May 4, 2026

Copy link
Copy Markdown
Contributor

Purpose

This PR is not meant to be merged. It exists only to validate the gitleaks workflow change in #32 (OPS-732) once that PR lands on `master`.

How to verify

  1. Merge fix(ci): [OPS-732] replace gitleaks-action with CLI for fork PR support #32 to `master`.
  2. Push any no-op commit to this branch (e.g. `git commit --allow-empty -m "ci: re-trigger" && git push`) or close + reopen this PR. Either re-fires the `pull_request_target` event, which now picks up the new YAML from `master`.
  3. Confirm the `gitleaks` check goes green and the run log shows:
    • `Install gitleaks` step succeeded (CLI version printed).
    • `Run gitleaks` step ran `gitleaks detect --source . --log-opts ..HEAD --exit-code 1` and exited 0.
    • No `The [pull_request_target] event is not yet supported` error.
  4. Close this PR without merging.

Limitations

  • This validates Path 2 (same-repo `pull_request_target`). Path 3 (fork PR) requires an actual fork; the workflow paths are identical except for ref-resolution, so a green Path 2 is strong evidence Path 3 will work.

Diff

Whitespace-only README tweak. No functional change.

🤖 Generated with Claude Code

Trivial whitespace tweak to README to generate a PR event. Used to
verify the gitleaks workflow change in OPS-732 (#32) once merged: after
that merge, push a no-op commit here to re-trigger CI and confirm the
gitleaks job runs the new CLI path instead of the rejected
gitleaks-action call. Close without merging once verified.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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