guettli/sharedinbox
1
0
Fork 0
Code Issues 21 Pull Requests 6 Actions Packages Projects Releases Wiki Activity

CI: ci.yml triggers full check on every push — add source path filters to skip doc/CI-only commits #493

New Issue
Closed
opened 2026-06-06 15:36:26 +00:00 by guettlibot · 0 comments
guettlibot commented 2026-06-06 15:36:26 +00:00 (Migrated from codeberg.org)
Copy Link
Copy Source

Problem

ci.yml triggers on all pushes to any branch with no path filtering:

on: [push, pull_request]

A commit that only changes docs, CI workflow files, the website, or scripts that aren't tested by the check suite still triggers the full 20–30 min task check-dagger run. This wastes runner time and delays unrelated CI jobs.

Fix

Add a paths filter so the check only runs when code that it actually tests changes:

on:
  push:
    paths:
      - 'lib/**'
      - 'test/**'
      - 'integration_test/**'
      - 'android/**'
      - 'linux/**'
      - 'pubspec.yaml'
      - 'pubspec.lock'
      - 'analysis_options.yaml'
      - 'drift_schemas/**'
      - 'ci/**'
      - 'scripts/**'
      - '.forgejo/workflows/ci.yml'
      - '.forgejo/Dockerfile'
  pull_request:
    paths:
      - 'lib/**'
      - 'test/**'
      - 'integration_test/**'
      - 'android/**'
      - 'linux/**'
      - 'pubspec.yaml'
      - 'pubspec.lock'
      - 'analysis_options.yaml'
      - 'drift_schemas/**'
      - 'ci/**'
      - 'scripts/**'
      - '.forgejo/workflows/ci.yml'
      - '.forgejo/Dockerfile'

Commits that only touch website/, docs/, *.md, or .forgejo/workflows/deploy.yml etc. will skip the check and be validated by the deploy or website workflow instead.

Expected impact

  • Eliminates full CI runs for doc-only / CI-config-only / website commits.
  • Faster merge cycles for non-code changes.
  • No loss of correctness — deploy.yml already has its own change detection.

Files to change

  • .forgejo/workflows/ci.yml
## Problem `ci.yml` triggers on **all** pushes to any branch with no path filtering: ```yaml on: [push, pull_request] ``` A commit that only changes docs, CI workflow files, the website, or scripts that aren't tested by the check suite still triggers the full 20–30 min `task check-dagger` run. This wastes runner time and delays unrelated CI jobs. ## Fix Add a `paths` filter so the check only runs when code that it actually tests changes: ```yaml on: push: paths: - 'lib/**' - 'test/**' - 'integration_test/**' - 'android/**' - 'linux/**' - 'pubspec.yaml' - 'pubspec.lock' - 'analysis_options.yaml' - 'drift_schemas/**' - 'ci/**' - 'scripts/**' - '.forgejo/workflows/ci.yml' - '.forgejo/Dockerfile' pull_request: paths: - 'lib/**' - 'test/**' - 'integration_test/**' - 'android/**' - 'linux/**' - 'pubspec.yaml' - 'pubspec.lock' - 'analysis_options.yaml' - 'drift_schemas/**' - 'ci/**' - 'scripts/**' - '.forgejo/workflows/ci.yml' - '.forgejo/Dockerfile' ``` Commits that only touch `website/`, `docs/`, `*.md`, or `.forgejo/workflows/deploy.yml` etc. will skip the check and be validated by the deploy or website workflow instead. ## Expected impact - Eliminates full CI runs for doc-only / CI-config-only / website commits. - Faster merge cycles for non-code changes. - No loss of correctness — `deploy.yml` already has its own change detection. ## Files to change - `.forgejo/workflows/ci.yml`
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
NeedSupervisor
Issue escalated to a human supervisor; agentloop will skip it until cleared.
 State/InProgress
State/Later
State/Planned
automerge
Eligible for automatic merge by CI
 ci-failure
Issue opened by agentloop to track a failing CI workflow; used for deduplication.
 do-not-merge
Plan PR — review only, do not merge.
 loop/code
Add to run the built-in "code" prompt; override at prompts/code.md.
 loop/code-ci-pending
Prompt "code" finished; waiting for the PR's CI to pass before advancing.
 loop/code-done
Prompt "code" finished successfully.
 loop/code-in-process
Agent for the "code" prompt is currently running on this issue.
 loop/merge
Managed by agentloop
 loop/merge-done
Managed by agentloop
 loop/merge-in-process
Managed by agentloop
 loop/plan
Add to run the built-in "plan" prompt; override at prompts/plan.md.
 loop/plan-done
Prompt "plan" finished successfully.
 loop/plan-in-process
Agent for the "plan" prompt is currently running on this issue.
No labels loop/plan-done
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
guettlibot guettli
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: guettli/sharedinbox#493