Skip to content

fix(ci): trigger deploy on main push and v* tag directly#8

Merged
enixCode merged 1 commit intomainfrom
fix/deploy-trigger
Apr 22, 2026
Merged

fix(ci): trigger deploy on main push and v* tag directly#8
enixCode merged 1 commit intomainfrom
fix/deploy-trigger

Conversation

@enixCode
Copy link
Copy Markdown
Owner

Summary

Listening on alpha/latest mobile tags broke the deploy pipeline after PR #6: release.yml pushes those tags with GITHUB_TOKEN, and tags pushed by a workflow do not trigger other workflows (GitHub anti-loop rule). So deploy.yml never ran.

Fix

Trigger on the original user pushes instead:

  • push: branches: [main] -> staging (light-process-test on lp-test.enixcode.fr)
  • push: tags: ['v*'] -> prod (light-process on lp.enixcode.fr)

Ternary on github.ref_type picks the script. No polling, no chain, 13 lines.

The mobile tags alpha / latest stay maintained by release.yml - still consumed by users for `npm install github:.../light-process#alpha`.

Test plan

  • YAML syntax valid
  • After merge: main push triggers deploy.yml -> staging redeploys (lp-test)
  • Next v* tag triggers deploy.yml -> prod redeploys (lp)

build with cc

Listening on the alpha/latest mobile tags broke the deploy: release.yml
pushes those tags with GITHUB_TOKEN, and a tag pushed by a workflow does
not trigger other workflows (GitHub anti-loop rule), so deploy.yml never
ran after the PR #6 merge.

Trigger on the original user pushes instead. Ternary on github.ref_type
picks the script: tag push -> prod (light-process), branch push -> staging
(light-process-test). No polling, no chain, 13 lines.

The alpha/latest mobile tags stay maintained by release.yml - they are
still used by consumers for `npm install github:.../light-process#alpha`.

build with cc
@enixCode enixCode merged commit f667acc into main Apr 22, 2026
3 checks passed
@enixCode enixCode deleted the fix/deploy-trigger branch April 22, 2026 05:50
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