Skip to content

Commit 8dab443

Browse files
committed
ci: fix Docker step — add -f docker/Dockerfile and drop invalid --reporter arg
The docker build step was looking for Dockerfile in repo root but it lives at docker/Dockerfile. Add -f flag to point at the correct path. Also remove --reporter compact from docker run; that is a Catch2 flag, not a ctest flag, and the container ENTRYPOINT is ctest. Signed-off-by: Matt Jones <matt@jellybaby.com> Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com>
1 parent 4fd2d43 commit 8dab443

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -377,10 +377,10 @@ jobs:
377377
- uses: actions/checkout@v4
378378

379379
- name: Build Docker image
380-
run: docker build --target test -t cpp-lin-test .
380+
run: docker build -f docker/Dockerfile --target test -t cpp-lin-test .
381381

382382
- name: Smoke test - run all tests in container
383-
run: docker run --rm cpp-lin-test --reporter compact
383+
run: docker run --rm cpp-lin-test
384384

385385
sarif:
386386
name: SARIF upload

0 commit comments

Comments
 (0)