Skip to content

Commit ed57d33

Browse files
committed
clean up unnecessary ecosystem check duplication
1 parent 118b499 commit ed57d33

File tree

2 files changed

+3
-69
lines changed

2 files changed

+3
-69
lines changed

.github/workflows/runtime-tests.yml

Lines changed: 3 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -3,60 +3,7 @@ name: Runtime Compatibility Tests
33
on: [push, pull_request]
44

55
jobs:
6-
runtimes:
7-
runs-on: ubuntu-latest
8-
strategy:
9-
matrix:
10-
os: [ubuntu-latest]
11-
node: [18, 20]
12-
include:
13-
- runner: deno
14-
- runner: bun
15-
16-
steps:
17-
- uses: actions/checkout@v4
18-
19-
# Setup Node.js for matrix versions
20-
- if: matrix.runner != 'deno' && matrix.runner != 'bun'
21-
uses: actions/setup-node@v4
22-
with:
23-
node-version: ${{ matrix.node }}
24-
25-
# Setup Deno
26-
- if: matrix.runner == 'deno'
27-
uses: denoland/setup-deno@v2
28-
with:
29-
deno-version: v1.x
30-
31-
# Setup Bun
32-
- if: matrix.runner == 'bun'
33-
uses: oven-sh/setup-bun@v2
34-
35-
- name: Install dependencies
36-
run: npm install
37-
38-
- name: Build
39-
run: npm run build
40-
41-
# Run runtime-specific tests
42-
- name: Test Node.js
43-
if: matrix.runner != 'deno' && matrix.runner != 'bun'
44-
run: npm run test:node
45-
46-
- name: Test Deno
47-
if: matrix.runner == 'deno'
48-
run: npm run test:deno
49-
50-
- name: Test Bun
51-
if: matrix.runner == 'bun'
52-
run: npm run test:bun
53-
54-
- name: Test Edge/Workers
55-
if: matrix.runner != 'deno' && matrix.runner != 'bun'
56-
run: npm run test:edge
57-
58-
# Fail-fast smoke tests (similar to OpenAI's ecosystem-tests)
59-
smoke-tests:
6+
runtime-compatibility:
607
runs-on: ubuntu-latest
618
steps:
629
- uses: actions/checkout@v4
@@ -73,15 +20,6 @@ jobs:
7320
npm install
7421
npm run build
7522
76-
- name: Quick compatibility check
77-
run: |
78-
# Node.js
79-
node -e "console.log('✅ Node CJS:', require('./lib/cjs/index.cjs').WorkOS.name)"
80-
node -e "import('./lib/esm/index.js').then(m => console.log('✅ Node ESM:', m.WorkOS.name))"
81-
82-
# Deno
83-
deno eval "import('./lib/esm/index.js').then(m => console.log('✅ Deno:', m.WorkOS.name))"
84-
85-
# Bun
86-
bun -e "console.log('✅ Bun:', require('./lib/cjs/index.cjs').WorkOS.name)"
23+
- name: Runtime compatibility check
24+
run: npm run check:runtimes
8725

package.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,7 @@
3737
"lint": "eslint",
3838
"test": "jest",
3939
"test:watch": "jest --watch",
40-
"test:worker": "jest src/worker.spec.ts",
4140
"check:runtimes": "tsx scripts/ecosystem-check.ts",
42-
"test:node": "node -e \"console.log('CJS:', require('./lib/cjs/index.cjs').WorkOS.name)\" && node -e \"import('./lib/esm/index.js').then(m => console.log('ESM:', m.WorkOS.name))\"",
43-
"test:deno": "deno eval \"import('./lib/esm/index.js').then(m => console.log('Deno:', m.WorkOS.name))\"",
44-
"test:bun": "bun -e \"console.log('Bun:', require('./lib/cjs/index.cjs').WorkOS.name)\"",
4541
"prettier": "prettier \"src/**/*.{js,ts,tsx}\" --check",
4642
"format": "prettier \"src/**/*.{js,ts,tsx}\" --write",
4743
"prepublishOnly": "npm run build"

0 commit comments

Comments
 (0)