Skip to content

ci: make required Frontend check report on every PR - #199

Merged
mergemaven11 merged 1 commit into
mainfrom
fix/frontend-required-check-all-prs
Aug 27, 2026
Merged

ci: make required Frontend check report on every PR#199
mergemaven11 merged 1 commit into
mainfrom
fix/frontend-required-check-all-prs

Conversation

@mergemaven11

Copy link
Copy Markdown
Owner

Problem

Test, lint and build frontend is now a globally required branch-protection check, but Frontend CI only triggers when frontend files change. Docs-only or backend-only PRs can therefore remain blocked waiting for a required check that never starts.

Fix

  • Trigger Frontend CI on every pull request so the required check always reports.
  • Detect whether the PR actually changes frontend/** or the Frontend CI workflow itself.
  • Run the full Node/browser/lint/build/bundle suite only for frontend-relevant PRs.
  • For non-frontend PRs, complete the same required job successfully after a lightweight change-detection step.
  • Keep push to main path-filtered so unnecessary post-merge frontend work is not added.

Why this preserves protection

The globally required job still exists and reports on every PR. Frontend changes still run the full existing validation suite, including the real-Chrome click audit. Non-frontend changes no longer wait indefinitely or spend several minutes running irrelevant browser tests.

Merge gate

Do not merge unless Backend, Frontend, and Security checks are green on this exact head, the branch is current with main, the PR is mergeable, and there are no unresolved blocker comments/reviews.

@mergemaven11
mergemaven11 force-pushed the fix/frontend-required-check-all-prs branch from e7731b4 to 3445a29 Compare August 27, 2026 02:20
@mergemaven11 mergemaven11 reopened this Aug 27, 2026
@mergemaven11
mergemaven11 merged commit 1cc307a into main Aug 27, 2026
3 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