Skip to content

ci: run tests on main pushes#40

Merged
danReynolds merged 1 commit into
mainfrom
codex/run-ci-on-main
May 31, 2026
Merged

ci: run tests on main pushes#40
danReynolds merged 1 commit into
mainfrom
codex/run-ci-on-main

Conversation

@danReynolds

Copy link
Copy Markdown
Owner

Summary

Adds a push trigger for main to the Flutter test workflow, while keeping the existing pull_request trigger for PRs targeting main.

Why

The workflow currently runs for PR updates only. After a PR is merged, the resulting commit on main does not start CI, so merge commits can land without a post-merge verification run.

Verification

  • git diff --check

Copilot AI review requested due to automatic review settings May 31, 2026 18:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Flutter test GitHub Actions workflow so CI runs not only for PRs targeting main, but also for commits pushed to main (e.g., after merges), ensuring post-merge verification.

Changes:

  • Add a push trigger for the main branch while keeping the existing pull_request trigger for PRs targeting main.
  • Minor YAML cleanup/formatting in the workflow file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@danReynolds danReynolds merged commit 82b8a7b into main May 31, 2026
2 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.

2 participants