Skip to content

ci(docker): self-trigger on workflow file changes#1550

Merged
brunobuddy merged 1 commit intomainfrom
ci/docker-self-trigger-path-filter
Apr 13, 2026
Merged

ci(docker): self-trigger on workflow file changes#1550
brunobuddy merged 1 commit intomainfrom
ci/docker-self-trigger-path-filter

Conversation

@brunobuddy
Copy link
Copy Markdown
Member

@brunobuddy brunobuddy commented Apr 13, 2026

Summary

Tiny follow-up to #1549. The Docker workflow's pull_request path filter didn't include .github/workflows/docker.yml itself, so PRs that only touch the workflow — like #1549 did — don't actually exercise the Docker validate job. I only caught this after #1549 merged and realised the new amd64/arm64 matrix had never been run against a real PR.

Adding the workflow file to its own path filter is standard practice and makes future workflow edits self-validate.

Test plan


Summary by cubic

Include .github/workflows/docker.yml in the Docker workflow’s pull_request.paths so edits to the workflow trigger the Docker validate job (amd64/arm64). This ensures workflow changes self-validate instead of silently skipping the matrix.

Written for commit 06aef89. Summary will update on new commits.

The path filter for pull_request didn't include the workflow file
itself, so PRs that only touch .github/workflows/docker.yml (like the
previous commit on this branch) don't exercise the Docker validate
job. Add the workflow file to its own filter so changes self-validate.
@brunobuddy brunobuddy merged commit 95b61b0 into main Apr 13, 2026
11 checks passed
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.93%. Comparing base (e1e07e3) to head (06aef89).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1550   +/-   ##
=======================================
  Coverage   97.93%   97.93%           
=======================================
  Files         118      118           
  Lines        8581     8581           
  Branches     3230     3230           
=======================================
  Hits         8404     8404           
  Misses        175      175           
  Partials        2        2           
Flag Coverage Δ
frontend 97.93% <ø> (ø)
shared 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@brunobuddy brunobuddy deleted the ci/docker-self-trigger-path-filter branch April 13, 2026 05:03
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