ci(drizzle): add informational CI workflow for drizzle-clickhouse#794
Draft
Copilot wants to merge 2 commits into
Draft
ci(drizzle): add informational CI workflow for drizzle-clickhouse#794Copilot wants to merge 2 commits into
Copilot wants to merge 2 commits into
Conversation
Copilot created this pull request from a session on behalf of
peter-leonov-ch
June 3, 2026 08:01
View session
|
|
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
| }) | ||
|
|
||
| it('refuses to bind null/undefined without an explicit type', () => { | ||
| expect(() => compile(sql`SELECT ${null}`)).toThrow(/cannot infer type/i) |
|
|
||
| it('refuses to bind null/undefined without an explicit type', () => { | ||
| expect(() => compile(sql`SELECT ${null}`)).toThrow(/cannot infer type/i) | ||
| expect(() => compile(sql`SELECT ${undefined}`)).toThrow(/cannot infer type/i) |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Copilot
AI
changed the title
Planning drizzle adapter implementation for ClickHouse
ci(drizzle): add informational CI workflow for drizzle-clickhouse
Jun 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The experimental
@clickhouse/drizzle-ormpackage (Phase 0 / MVP) had no dedicated CI signal, so there was no easy way to see which checks pass and which fail as the adapter evolves. This adds a non-blocking workflow that runs the package's checks and reports per-check status..github/workflows/drizzle.yml— informational, non-blocking workflow onworkflow_dispatchplus pushes/PRs touchingpackages/drizzle-clickhouse/**or the workflow file. Runs build → typecheck → lint → unit tests across a Node 20/22/24 matrix. Each check usescontinue-on-error: trueand writes a pass/fail table to the job summary, so it surfaces results without gating PRs. Follows repo conventions (permissions: {}, pinned action SHAs,concurrencycancel-in-progress,fail-fast: false).test:drizzle:unitroot npm script — runs the drizzle unit tests in isolation (TEST_MODE=unit vitest run … packages/drizzle-clickhouse), reused by the workflow.Note: the drizzle unit tests already run in the main
testsworkflow vianode-unit-tests(TEST_MODE=unit); this workflow is a focused, always-visible dashboard rather than a replacement. Droppingcontinue-on-errorand adding an aggregate gate would turn it into a required check if desired.Checklist