Skip to content

ci: run Flutter check on every PR so it can be required - #41

Merged
DCT-Berinyuy merged 1 commit into
devfrom
ci-required-check-all-prs
Sep 27, 2026
Merged

DCT-Berinyuy merged 1 commit into
devfrom
ci-required-check-all-prs

Conversation

@DCT-Berinyuy

Copy link
Copy Markdown
Collaborator

dev now requires the Analyze, test, build Android check (branch protection, admins included).

The pull_request trigger had paths-ignore for supabase/**, bookbridge-rust-core/**, landingPage/** and **.md. A required check that never runs leaves the PR stuck on "Expected — waiting", so PRs touching only those paths (e.g. the upcoming RLS-policy migrations) could never merge.

  • Removes paths-ignore from pull_request only; push filtering is unchanged.
  • Cost: those PRs now also run the Flutter build.

Test plan: this PR itself must show the check running and passing.

dev now requires 'Analyze, test, build Android'. With paths-ignore on
pull_request, PRs touching only supabase/, bookbridge-rust-core/,
landingPage/ or docs never report the check and cannot be merged.

Co-authored-by: Copilot App <[email protected]>
@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
book-bridge Ready Ready Preview Sep 27, 2026 8:44pm UTC

@DCT-Berinyuy
DCT-Berinyuy merged commit a65a993 into dev Sep 27, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — caea72e5 Deployed Sep 27, 2026 by vercel[bot]
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