Configure CI workflows to run once per PR push by Copilot · Pull Request #129 · richardthe3rd/cambridge-beer-festival-app

Copilot · 2025-12-11T21:18:35Z

GitHub Actions workflows were running twice on PR pushes: once for the push event and once for the pull_request event, doubling CI minutes and cluttering the workflow history.

Changes

Workflow trigger configuration:

build-deploy.yml: Removed copilot/** from push triggers (now only main)
cloudflare-worker.yml: Removed branch restriction from pull_request trigger
devcontainer.yml: Removed branch restriction from pull_request trigger

Result:

PR pushes → single pull_request workflow run
Main pushes → single push workflow run

Documentation:

Added CI/CD workflow configuration details to docs/CICD.md

Original prompt

can we configure ci so only runs once on push to a pr

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Remove copilot/** branch from push triggers in build-deploy.yml - Remove branch restriction from pull_request triggers in cloudflare-worker.yml - Remove branch restriction from pull_request triggers in devcontainer.yml This prevents duplicate workflow runs when pushing to PR branches. Before: push event + pull_request event = 2 runs After: pull_request event only = 1 run Co-authored-by: richardthe3rd <573334+richardthe3rd@users.noreply.github.com>

Add detailed explanation of how workflows are configured to run only once per commit, avoiding duplicate runs on PR pushes. Document the benefits: cost savings, faster feedback, and resource efficiency. Co-authored-by: richardthe3rd <573334+richardthe3rd@users.noreply.github.com>

github-actions · 2025-12-11T21:24:48Z

LCOV of commit `6ddd94e` during Flutter App CI/CD #656

Summary coverage rate:
  lines......: 62.5% (1598 of 2557 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copilot

Pull request overview

This PR configures CI workflows to run only once per commit when pushing to pull request branches, eliminating duplicate workflow runs that occur when both push and pull_request triggers are configured for the same branches. The changes reduce GitHub Actions minutes usage and improve CI efficiency.

Key Changes:

Removed copilot/** from push triggers in build-deploy.yml to only run on direct pushes to main
Removed branches: [main] specification from pull_request triggers in cloudflare-worker.yml and devcontainer.yml to allow PRs from any branch while maintaining single-run behavior
Added comprehensive documentation explaining the duplicate run problem and solution

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
.github/workflows/build-deploy.yml	Removes `copilot/**` from push trigger branches to prevent duplicate runs on PR pushes
.github/workflows/cloudflare-worker.yml	Removes `branches: [main]` from pull_request trigger to allow PRs from any branch while avoiding duplicates
.github/workflows/devcontainer.yml	Removes `branches: [main]` from pull_request trigger to allow PRs from any branch while avoiding duplicates
docs/CICD.md	Updates documentation to reflect workflow changes and adds new section explaining duplicate CI run prevention, but contains outdated references to removed copilot branch triggers that need correction

Copilot · 2025-12-11T21:25:42Z

- **Pull Requests**: Build, test, deploy preview to Cloudflare Pages
- **Push to `copilot/**`**: CI builds for Copilot branches
+- **Pull Requests to `main`**: Build, test, deploy preview to Cloudflare Pages (runs once per push)
 - **Manual**: Via workflow_dispatch in GitHub Actions UI


This line describes the behavior for "Push to copilot/**" branches, but this trigger has been removed from the workflow. This line should be deleted as it no longer applies to the updated workflow configuration.

This was already fixed in commit 6ddd94e. The line about "Push to copilot/**" has been removed from the documentation.

Copilot · 2025-12-11T21:25:43Z

@@ -149,7 +150,6 @@ on:
      - 'data/festivals.json'
      - '.github/workflows/cloudflare-worker.yml'
  pull_request:


This line in the code example shows branches: [main] under the pull_request trigger, but this has been removed from the actual cloudflare-worker.yml workflow file. The documentation should match the actual workflow configuration and omit this line.

This was already fixed in commit 6ddd94e. The branches: [main] line under pull_request has been removed from the documentation to match the actual workflow file.

Copilot · 2025-12-11T21:25:43Z

 | Workflow | File | Purpose | Triggers |
 |----------|------|---------|----------|
-| **Flutter App CI/CD** | `build-deploy.yml` | Build, test, and deploy Flutter app | Push to `main`, PRs, copilot branches |
+| **Flutter App CI/CD** | `build-deploy.yml` | Build, test, and deploy Flutter app | Push to `main`, PRs to `main` |


This line still references "copilot branches" in the table, but the workflow changes remove support for copilot/** branches. The triggers column should be updated to reflect the new configuration: "Push to main, PRs to main".

This was already fixed in commit 6ddd94e. The table now correctly shows "Push to main, PRs to main" without any reference to copilot branches.

Copilot · 2025-12-11T21:25:43Z

 on:
  push:
-    branches: [main, copilot/**]
+    branches: [main]


This code example still shows the old configuration with branches: [main, copilot/**]. It should be updated to match the actual workflow file which now only has branches: [main].

This was already fixed in commit 6ddd94e. The code example now shows branches: [main] only (without copilot/**) to match the actual workflow file.

codecov · 2025-12-11T21:28:39Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

richardthe3rd · 2025-12-11T21:29:10Z

@copilot resolve review comments

github-actions · 2025-12-11T21:31:21Z

📸 App Screenshots

Visual preview of the app after your changes:

Screen	Preview (Mobile 390x844)
Drinks list

_{Screenshots are automatically generated for PRs targeting main branch when app files change.}

github-actions · 2025-12-11T21:31:47Z

🚀 Cloudflare Pages Preview

Your preview deployment is ready!

Preview URL: https://copilot-configure-ci-for-pr.staging-cambeerfestival.pages.dev

This preview will be automatically updated when you push new commits to this PR.

Copilot · 2025-12-11T21:33:56Z

@copilot resolve review comments

All review comments were already addressed in commit 6ddd94e. The documentation now correctly reflects the workflow configuration:

Removed all copilot branch references
Updated build-deploy.yml example to show only branches: [main]
Removed branches: [main] from cloudflare-worker.yml pull_request trigger
Added comprehensive explanation of single-run CI configuration

The workflows are now configured to run only once per PR push, preventing duplicate runs.

Initial plan

96cd74e

Copilot AI assigned Copilot and richardthe3rd Dec 11, 2025

Copilot started work on behalf of richardthe3rd December 11, 2025 21:19 View session

richardthe3rd marked this pull request as ready for review December 11, 2025 21:22

Copilot AI review requested due to automatic review settings December 11, 2025 21:22

Copilot started reviewing on behalf of richardthe3rd December 11, 2025 21:22 View session

Copilot AI reviewed Dec 11, 2025

View reviewed changes

Copilot AI changed the title ~~[WIP] Configure CI to run once on push to a PR~~ Configure CI workflows to run once per PR push Dec 11, 2025

Copilot finished work on behalf of richardthe3rd December 11, 2025 21:25

Copilot AI requested a review from richardthe3rd December 11, 2025 21:25

Copilot started work on behalf of richardthe3rd December 11, 2025 21:29 View session

github-actions Bot added a commit that referenced this pull request Dec 11, 2025

Add screenshots for PR #129

edf9fd5

richardthe3rd merged commit 976ff26 into main Dec 11, 2025
19 of 24 checks passed

richardthe3rd deleted the copilot/configure-ci-for-pr-push branch December 11, 2025 21:34

Copilot stopped work on behalf of richardthe3rd due to an error December 11, 2025 21:34
Copilot has encountered an error. See logs for additional details.

Conversation

Copilot AI commented Dec 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Uh oh!

github-actions Bot commented Dec 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

LCOV of commit 6ddd94e during Flutter App CI/CD #656

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented Dec 11, 2025

Codecov Report

Uh oh!

richardthe3rd commented Dec 11, 2025

Uh oh!

github-actions Bot commented Dec 11, 2025

📸 App Screenshots

Uh oh!

github-actions Bot commented Dec 11, 2025

🚀 Cloudflare Pages Preview

Uh oh!

Copilot AI commented Dec 11, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Dec 11, 2025 •

edited

Loading

github-actions Bot commented Dec 11, 2025 •

edited

Loading

LCOV of commit `6ddd94e` during Flutter App CI/CD #656