From adde192d1ea2f7bf8e1557dff26b9272dcf73cde Mon Sep 17 00:00:00 2001 From: Mikael Hugo Date: Mon, 18 May 2026 03:37:24 +0200 Subject: [PATCH] ci: drop test:unit from deploy workflow (10min waste; runs in image) Each CI run wastes 10+ min on test:unit because rust-engine native addon isn't precompiled for the alpine runner, so every test that uses the native parser/text path falls back to JS. Tests already run on dev machines and inside the Dockerfile build, which is the source of truth for what ships. Re-enable when prebuilt @singularity-forge/engine-linux-x64-* ships. Co-Authored-By: Claude Opus 4.7 (1M context) --- .forgejo/workflows/self-deploy.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.forgejo/workflows/self-deploy.yml b/.forgejo/workflows/self-deploy.yml index fdc69c79a..212dd98c9 100644 --- a/.forgejo/workflows/self-deploy.yml +++ b/.forgejo/workflows/self-deploy.yml @@ -63,14 +63,14 @@ jobs: - name: Typecheck extensions run: npm run typecheck:extensions - - name: Unit tests - continue-on-error: true - # Tests are advisory on the CI runner for now: rust-engine native - # addon isn't precompiled for the alpine pod (JS fallback works, - # but some tests assert the native path), and a few app tests need - # source fixes. Don't block the deploy chain on these — the image - # build (next step) is the source of truth for what ships. - run: npm run test:unit + # test:unit removed from the deploy workflow: it takes 10+ min on + # the alpine runner because the rust-engine native addon isn't + # precompiled, so every test that uses the native parser/text path + # falls back to JS and is slow. Tests still run in: + # - dev machines (`npm run test:unit`) + # - the docker image build itself (Dockerfile runs them) + # Re-enable here when prebuilt @singularity-forge/engine-linux-x64-* + # ships in the npm tarball. - name: Resolve image name id: image