Skip to content

Commit 6c727b9

Browse files
committed
ci(lint): drop tsc-tests job; keep clippy only
tests-sdk depends on @lazorkit/sdk-legacy via a 'file:' link to the sibling lazorkit-protocol checkout. CI checks out only this repo, so the dep can't resolve and tsc fails on every import. Switch to the npm-published version (^0.3.1+) once it lands, then restore the tsc job. Type errors still caught locally.
1 parent b25519c commit 6c727b9

1 file changed

Lines changed: 9 additions & 20 deletions

File tree

.github/workflows/lint.yml

Lines changed: 9 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -44,23 +44,12 @@ jobs:
4444
-A clippy::too_many_arguments \
4545
-A unused_mut
4646
47-
tsc:
48-
name: tsc (tests-sdk)
49-
runs-on: ubuntu-latest
50-
timeout-minutes: 5
51-
steps:
52-
- uses: actions/checkout@v4
53-
54-
- uses: actions/setup-node@v4
55-
with:
56-
node-version: 20
57-
cache: npm
58-
cache-dependency-path: tests-sdk/package-lock.json
59-
60-
- name: install
61-
working-directory: tests-sdk
62-
run: npm ci || npm install
63-
64-
- name: tsc --noEmit
65-
working-directory: tests-sdk
66-
run: npx tsc --noEmit
47+
# tsc on tests-sdk is intentionally NOT included here.
48+
#
49+
# tests-sdk depends on `@lazorkit/sdk-legacy` via a `file:` link to the
50+
# sibling lazorkit-protocol checkout, which doesn't exist in CI. Wiring
51+
# this up cleanly requires switching to the npm-published version
52+
# (^0.3.1+) once it lands, then restoring the tsc job.
53+
#
54+
# Type errors are still caught locally via `npx tsc --noEmit` from
55+
# tests-sdk against the file: linked SDK.

0 commit comments

Comments
 (0)