Repository navigation
Add Bun smoke test to CI - #1482
cristianrgreco wants to merge 1 commit into
Conversation
✅ Deploy Preview for testcontainers-node ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe checks workflow adds a Bun smoke-test job when the testcontainers module is detected. The job builds and prunes the workspace, installs Bun 1.4.2, and runs CommonJS and ES module smoke tests with Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The Bun smoke-test job adds CI coverage without an identified merge-blocking risk. Merge after normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Adds a
Smoke tests (Bun)CI job that runs the existing CommonJS and ES module smoke tests (start and stop analpinecontainer) under Bun 1.4.2 (the current latest release; pinned so Bun releases do not break unrelated PRs, to be bumped periodically).The job mirrors the Node.js smoke test job: build
testcontainerswith Node.js, prune dev dependencies, then runpackages/testcontainers/smoke-test.jsandpackages/testcontainers/smoke-test.mjswithbun. Like the Node.js smoke tests, it only runs when thetestcontainerspackage changes, and theChecks completejob depends on it.This replaces the full Bun test matrix proposed in #1433 with a lightweight compatibility signal.
Verification
npm run formatnpm run lint.github/workflows/checks.ymland confirmed the new job andenddependenciesnpm run buildandnpm prune --omit=dev --workspace packages/testcontainers:bun packages/testcontainers/smoke-test.js: exit 0bun packages/testcontainers/smoke-test.mjs: exit 0Compatibility
Not breaking: CI-only change. No published package code or public API changes.