Skip to content

fix: use workflow_run trigger in rebuild-site.yml to bypass [skip ci] suppression#60

Merged
davidamitchell merged 1 commit into
mainfrom
copilot/review-latest-pipeline-logs
May 30, 2026
Merged

fix: use workflow_run trigger in rebuild-site.yml to bypass [skip ci] suppression#60
davidamitchell merged 1 commit into
mainfrom
copilot/review-latest-pipeline-logs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 29, 2026

rebuild-site.yml has never fired from the daily pipeline. Bot commits use [skip ci] in the message, which GitHub uses to suppress all push-triggered workflows — including the push: paths: data/processed/** trigger. docs/data/*.json has never been regenerated; the live site has been serving placeholder JSON since launch.

Change

Replace the push trigger with workflow_run, which fires on workflow completion events and is immune to [skip ci]:

# Before
on:
  push:
    branches: [main]
    paths:
      - "data/processed/**"

# After
on:
  workflow_run:
    workflows: ["Pipeline", "Fetch and Process"]
    types: [completed]
    branches: [main]

Add a job-level guard so the rebuild is skipped when the upstream workflow failed:

if: ${{ github.event_name == 'workflow_dispatch' || github.event.workflow_run.conclusion == 'success' }}

The [skip ci] markers on bot commits are intentional and unchanged — this fix only corrects where the site-rebuild trigger listens.

Co-authored-by: davidamitchell <635733+davidamitchell@users.noreply.github.com>
Copilot AI changed the title fix: replace push trigger with workflow_run in rebuild-site.yml fix: use workflow_run trigger in rebuild-site.yml to bypass [skip ci] suppression May 29, 2026
Copilot AI requested a review from davidamitchell May 29, 2026 20:13
@davidamitchell davidamitchell marked this pull request as ready for review May 30, 2026 03:09
@davidamitchell davidamitchell merged commit 6adc0ba into main May 30, 2026
1 check passed
@davidamitchell davidamitchell deleted the copilot/review-latest-pipeline-logs branch May 30, 2026 03:09
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