Skip to content

Split test workflow into PR and main branch push#59

Merged
jovnc merged 2 commits intogit-mastery:mainfrom
jovnc:test/split-workflow
Mar 7, 2026
Merged

Split test workflow into PR and main branch push#59
jovnc merged 2 commits intogit-mastery:mainfrom
jovnc:test/split-workflow

Conversation

@jovnc
Copy link
Collaborator

@jovnc jovnc commented Mar 7, 2026

Separates the single test.yml workflow (which used two jobs with if: guards on github.event_name) into two dedicated files:

  1. test-pr.yml: runs on pull_request_target with the e2e-test environment gate to protect secrets from untrusted PR code
  2. test.yml: runs on push to main and workflow_dispatch

This makes the intent of each workflow clearer and removes the need for runtime if: conditions to differentiate between event types.

@jovnc jovnc merged commit 2670d80 into git-mastery:main Mar 7, 2026
1 of 4 checks passed
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