Skip to content

Upgrade SQL kernel native packages to 1.0.0 - #516

Merged
vuanhphung merged 1 commit into
mainfrom
vu-phung/upgrade-sql-kernel-1.0.0
Sep 1, 2026
Merged

Upgrade SQL kernel native packages to 1.0.0#516
vuanhphung merged 1 commit into
mainfrom
vu-phung/upgrade-sql-kernel-1.0.0

Conversation

@vuanhphung

Copy link
Copy Markdown
Collaborator

Upgrade all platform-specific databricks-sql-kernel packages from 0.2.0 to 1.0.0 and refresh their lockfile metadata through npm-proxy.cloud.databricks.com.

This also updates the napi-rs router version guard and release documentation.

Validation: npm run build; 329 kernel tests against the installed 1.0.0 binary; 1,290 full unit tests; Prettier and git diff --check.


This PR was created with GitHub MCP.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase (git rebase -i main).

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: 1 High

Mechanical version bump of the kernel native packages 0.2.0 → 1.0.0 across index.js version guards, package.json, and the lockfile — the code changes are consistent and correct. One high concern: the lockfile resolved URLs now point at the internal npm-proxy.cloud.databricks.com host rather than the public npm registry, which will break npm ci for external consumers/CI.

Comment thread package-lock.json Outdated
Signed-off-by: Vu Anh Phung <vu.phung@databricks.com>
@vuanhphung
vuanhphung force-pushed the vu-phung/upgrade-sql-kernel-1.0.0 branch from ed14fa3 to 7b7dad4 Compare September 1, 2026 03:03

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No issues identified by the review bot.

@vuanhphung
vuanhphung added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit a0d7625 Sep 1, 2026
25 checks passed
@vuanhphung
vuanhphung deleted the vu-phung/upgrade-sql-kernel-1.0.0 branch September 1, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants