Skip to content

fix(actions): use repository check instead of event_name for dogfooding conditions#727

Merged
BYK merged 1 commit intomasterfrom
fix/reusable-workflow-build-condition
Jan 22, 2026
Merged

fix(actions): use repository check instead of event_name for dogfooding conditions#727
BYK merged 1 commit intomasterfrom
fix/reusable-workflow-build-condition

Conversation

@BYK
Copy link
Member

@BYK BYK commented Jan 22, 2026

Summary

When external repos call the reusable release workflow via workflow_call, the github.event_name still reflects the original trigger (e.g., workflow_dispatch from the caller). This caused the build job and dogfooding steps to incorrectly run for external repos.

This fix changes the conditions to check github.repository == 'getsentry/craft' instead of github.event_name, properly distinguishing Craft's own releases from external callers.

Changes

  • Build job: github.event_name == 'workflow_dispatch'github.repository == 'getsentry/craft'
  • Dogfooding step: github.event_name == 'workflow_dispatch'github.repository == 'getsentry/craft'
  • External step: github.event_name == 'workflow_call'github.repository != 'getsentry/craft'

…ions

When external repos call the reusable workflow via workflow_call, the
github.event_name still reflects the original trigger (e.g. workflow_dispatch).
This caused the build job and dogfooding steps to run for external repos.

Use github.repository check instead to properly distinguish Craft's own
releases from external callers.
@BYK BYK changed the title fix: use repository check instead of event_name for dogfooding conditions fix(actions): use repository check instead of event_name for dogfooding conditions Jan 22, 2026
@BYK BYK enabled auto-merge (squash) January 22, 2026 01:23
BYK added a commit to getsentry/cli that referenced this pull request Jan 22, 2026
## Summary

The reusable Craft workflow has a bug where it runs Craft's internal
build/test jobs for external callers (see getsentry/craft#727).

Use the `getsentry/craft@v2` action directly instead, like sentry-python
does.
@BYK BYK merged commit 9125998 into master Jan 22, 2026
18 checks passed
@BYK BYK deleted the fix/reusable-workflow-build-condition branch January 22, 2026 06:42
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.

2 participants