Skip to content

Commit cbe5c30

Browse files
roypatstefano-garzarella
authored andcommitted
ci: drop path filtering from pull_request target
Since the "docker container builds" sanity check is required for merging (so that we can use auto-merge), it needs to always trigger, so drop the filtering based on which files were modified. There's the slight downside that it means we'll need to wait 3+ hours even on PRs that dont actually touch the container, but those are very rare from experience. If it ever becomes a nuisance, we can modify the action to instead do the paths check inside the actual command step, and return early. Leave the check in the `push` target, so that we do not unnecessarily build docker containers after merging though. Signed-off-by: Patrick Roy <[email protected]>
1 parent 6547482 commit cbe5c30

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

.github/workflows/docker-publish.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,6 @@ on:
1212
- riscv64/*
1313
pull_request:
1414
branches: [ "main" ]
15-
paths:
16-
- Dockerfile
17-
- .github/workflows/docker-publish.yml
18-
- build_container.sh
19-
- Dockerfile.riscv64
20-
- riscv64/*
2115

2216
jobs:
2317
build:

0 commit comments

Comments
 (0)