From fe6749d3af604386b0a7cd96aa75e273d1307126 Mon Sep 17 00:00:00 2001 From: Ivan FB Date: Fri, 12 Jun 2026 14:55:38 +0200 Subject: [PATCH] ci(test): pass -d:ffiAllowSignalHandler when compiling unit tests The noSignalHandler compile-time guard rejects any build of the ffi modules that doesn't set -d:noSignalHandler (embedded) or -d:ffiAllowSignalHandler (standalone). test.yml compiled the unit tests with a raw `nim c` missing the flag, so every per-test job failed at the guard. The nimble test tasks already pass it; mirror that here. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index c009658..7729732 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -83,4 +83,4 @@ jobs: if [ "$RUNNER_OS" == "Windows" ]; then export PATH="$GITHUB_WORKSPACE/.nim_runtime/bin:$HOME/.nimble/bin:$PATH" fi - nim c -r --mm:${{ matrix.mm }} -d:chronicles_log_level=WARN tests/unit/${{ inputs.test }}.nim + nim c -r --mm:${{ matrix.mm }} -d:chronicles_log_level=WARN -d:ffiAllowSignalHandler tests/unit/${{ inputs.test }}.nim