Skip to content

Commit

Permalink
Temporary disable tests on ubuntu-latest and macOS-latest for Bun
Browse files Browse the repository at this point in the history
  • Loading branch information
adams85 committed Nov 21, 2024
1 parent ff6f6e2 commit 6fbefa3
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/js-sdk-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,9 @@ jobs:
strategy:
matrix:
bun-version: ["1.1.0", "latest"]
os: [ubuntu-latest, windows-latest, macOS-latest]
# Bun fails to run tests on ubuntu-latest and macOS-latest.
# It may not be possible at all for now: https://github.com/oven-sh/bun/issues/10001
os: [windows-latest]
name: Bun ${{ matrix.bun-version }} (${{ matrix.os }}) test
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -239,7 +241,7 @@ jobs:
needs: [
check-ts-compat,
test-browser-chrome, test-browser-chromium, test-browser-firefox,
test-bun,
test-bun #,
test-chromium-extension-chrome, test-chromium-extension-chromium,
test-cloudflare-worker,
test-deno,
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ The SDK is [tested](https://github.com/configcat/js-unified-sdk/blob/master/.git
- Chromium (64.0.3282.0, 72.0.3626.0, 80.0.3987.0)
- Firefox (latest, latest-beta, 84.0)
- @configcat/sdk/bun:
- Bun (v1.1.0, latest stable) on Windows / Ubuntu / macOS
- Bun (v1.1.0, latest stable) on Windows
- @configcat/sdk/deno:
- Deno (v1.31, v1.46, latest stable) on Windows / Ubuntu / macOS
- @configcat/sdk/node:
Expand Down
2 changes: 1 addition & 1 deletion test/bun/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,6 @@ for (const file of glob.globIterateSync(normalizePathSeparator(testDir) + "/**/*
segments[segments.length - 1] = path.basename(fileName, path.extname(fileName));

/* eslint-disable @typescript-eslint/no-require-imports */
require("../" + segments.join("/"));
await import("../" + segments.join("/"));
}
}

0 comments on commit 6fbefa3

Please sign in to comment.