Skip to content

fix(ci): skip Datadog reporting steps on non-main branches#63

Merged
platinummonkey merged 1 commit intomainfrom
fix/ci-skip-dd-reporting-on-forks
Feb 14, 2026
Merged

fix(ci): skip Datadog reporting steps on non-main branches#63
platinummonkey merged 1 commit intomainfrom
fix/ci-skip-dd-reporting-on-forks

Conversation

@platinummonkey
Copy link
Collaborator

Summary

  • Gate DD CI reporting steps (STS credentials, test optimization, orchestrion, coverage upload) to only run on main
  • Fixes forked PR CI failures caused by dd-sts-action requiring id-token:write permissions unavailable to forks

Changes

  • Add if: github.ref == 'refs/heads/main' to "Get Datadog credentials", "Configure Datadog Test Optimization", "Install Datadog CI tools", and "Upload coverage to Datadog" steps
  • Tests still run on all PRs via the existing plain go test fallback when DD_API_KEY is unset

Test plan

  • Verify CI passes on this PR (non-main branch) without hitting STS permission errors
  • Verify DD reporting still works on pushes to main

🤖 Generated with Claude Code

@platinummonkey platinummonkey requested a review from a team as a code owner February 14, 2026 19:31
Forked PRs lack id-token:write permissions required by dd-sts-action,
causing the "Get Datadog credentials" step to fail. Gate all DD CI
reporting steps (STS credentials, test optimization, orchestrion,
coverage upload) to only run on main. Tests still run on all PRs via
the existing plain `go test` fallback.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@platinummonkey platinummonkey force-pushed the fix/ci-skip-dd-reporting-on-forks branch from 1dfd185 to 064a6c8 Compare February 14, 2026 19:34
@platinummonkey platinummonkey merged commit 551ef30 into main Feb 14, 2026
5 checks passed
@platinummonkey platinummonkey deleted the fix/ci-skip-dd-reporting-on-forks branch February 14, 2026 19:34
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