From 2385b7ded49b496a40ffb6429222a5b252ebe42e Mon Sep 17 00:00:00 2001 From: dantio Date: Wed, 29 Jul 2026 22:42:01 +0200 Subject: [PATCH] ci: run lint and typecheck, and trigger on pushes to master MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The workflow only ran `npm test`, and only on pull requests. Lint had no CI protection at all, and nothing type-checked test/ or examples/ — which is how an extensionless import survived in examples/traditional/clientAlerts.GetPublicAlerts.ts until it was caught by hand during the v10 review. Adds the lint and typecheck steps, and a push trigger scoped to master so merges and any direct pushes are verified too. The push filter is limited to master, so PR branches still fire only the pull_request event and nothing runs twice. --- .github/workflows/test.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 8662bd9b..e309cdae 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -3,6 +3,8 @@ name: Test on: pull_request: branches: [master] + push: + branches: [master] jobs: test: @@ -14,4 +16,6 @@ jobs: node-version: 22 cache: npm - run: npm ci + - run: npm run lint + - run: npm run typecheck - run: npm test