Skip to content

fix: remove event_name check from push-to-gitlab job condition - #23

Closed
surrealwolf wants to merge 0 commit into
mainfrom
fix/push-to-gitlab-condition
Closed

surrealwolf wants to merge 0 commit into
mainfrom
fix/push-to-gitlab-condition

Conversation

@surrealwolf

Copy link
Copy Markdown
Collaborator

The 'Push to GitLab' job was being skipped on main branch when the workflow was triggered by workflow_dispatch or other events (not just push).

Issue:
Job condition required github.event_name == 'push', so it only ran on direct pushes, not manual triggers.

Changes:

  • Removed github.event_name == 'push' requirement from job condition
  • Job now runs on main branch when tests pass, regardless of trigger type
  • Condition: if: always() && needs.test.result == 'success' && github.ref == 'refs/heads/main'

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