An E2E test for @clickhouse/client package beta version#1553
An E2E test for @clickhouse/client package beta version#1553peter-leonov-ch wants to merge 2 commits intomainfrom
@clickhouse/client package beta version#1553Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR Review✅ No critical issues found. Notes:
Recommendations for testing:
Since this is a test PR for QA purposes, CI validation is the primary concern. |
E2E Test Results✅ All tests passed • 86 passed • 3 skipped • 968s
Tests ran across 4 shards in parallel. |
a465b8e to
7cdb8fc
Compare
ae508f4 to
e15c096
Compare
@clickhouse/client package beta version
e15c096 to
0334d2e
Compare
There was a problem hiding this comment.
Code Review
⚠️ Note: The PR description explicitly states DO NOT MERGE — this is an E2E compatibility test for the@clickhouse/clientrelease cycle.
Changes reviewed: packages/common-utils/package.json and yarn.lock — bumping @clickhouse/client, @clickhouse/client-common, and @clickhouse/client-web from ^1.12.1 → beta (resolves to 1.18.2).
- ❌
"beta"is a floating dist-tag, not a pinned version → if/when merged to production, replace with an explicit version (e.g."^1.18.2") to avoid surprise breakage from future beta publishes ⚠️ yarn.lockresolution looks correct (all three packages resolve to1.18.2) — no mismatch issues
✅ No logic changes; the only code touched is the dependency version specifier and the generated lockfile. Acceptable as a temporary CI compatibility test PR as intended.


Warning
DO NOT MERGE
This PR is not meant to be merged, it's an e2e test for the
@clickhouse/clientpackage release cycleWhy
HDX is a heavy CH JS client user. For the mutual benefit let's test CH JS releases as part of the CH JS QA process.
What
Bumping the client version and checking the CI checks.
CH JS team is going to update this PR with beta versions of CH JS package during the release cycle.