Skip to content

devops: scope staging deploy trigger to main, skip docs-only PRs - #319

Merged
Austinaminu2 merged 1 commit into
FlowwStar:mainfrom
The-moving-man:fix/staging-workflow-branch-filter
Jul 24, 2026
Merged

devops: scope staging deploy trigger to main, skip docs-only PRs#319
Austinaminu2 merged 1 commit into
FlowwStar:mainfrom
The-moving-man:fix/staging-workflow-branch-filter

Conversation

@The-moving-man

@The-moving-man The-moving-man commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • staging.yml's pull_request trigger had no branches: filter (unlike ci.yml, which restricts to [main]), so every PR — including docs-only ones and PRs against non-main branches — triggered a Vercel staging deploy.
  • Adds branches: [main] to match ci.yml's pattern, plus paths-ignore: ['**/*.md'] to skip docs-only changes.

Fixes #306
closes #304 , closes #305 , closes #306 , closes #307

Test plan

  • Verify a PR targeting a non-main branch no longer triggers the staging workflow
  • Verify a docs-only PR (.md changes) targeting main no longer triggers the staging workflow
  • Verify a normal code PR targeting main still triggers the staging deploy as before

Every PR to any branch was triggering a Vercel staging deploy since
the pull_request trigger had no branches filter, unlike ci.yml. Add
branches: [main] to match ci.yml, and paths-ignore for markdown-only
changes to avoid unnecessary deploys.

Fixes FlowwStar#306

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@drips-wave

drips-wave Bot commented Jul 24, 2026

Copy link
Copy Markdown

@The-moving-man Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Austinaminu2
Austinaminu2 merged commit 6bfcf6e into FlowwStar:main Jul 24, 2026
1 of 6 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

2 participants