Skip to content

Update README setup instructions#1911

Merged
rugpanov merged 2 commits into
mainfrom
docs/update-readme-setup
Jun 16, 2026
Merged

Update README setup instructions#1911
rugpanov merged 2 commits into
mainfrom
docs/update-readme-setup

Conversation

@rugpanov

Copy link
Copy Markdown
Contributor

Why

The setup steps in the root README.md were stale: they pinned yarn v2 while package.json declares yarn@3.2.1, and they referenced an install:sdk step that no longer exists.

What

  • Bump npm install -g yarn@2yarn@3 to match packageManager in package.json.
  • Drop the obsolete yarn run install:sdk step. @databricks/sdk-experimental is now a regular npm dependency installed automatically by yarn install.

Verification

  • Confirmed no install:sdk script remains in any package.json.
  • Confirmed @databricks/sdk-experimental is a dependency in packages/databricks-vscode/package.json.
  • Confirmed packageManager is yarn@3.2.1.

This pull request and its description were written by Isaac.

*Why*:
* The setup steps were stale: they pinned yarn v2 while the repo's package.json declares yarn@3, and they referenced an `install:sdk` step that no longer exists.

*What:*
* Bump `npm install -g yarn@2` to `yarn@3` to match `packageManager` in package.json.
* Drop the obsolete `yarn run install:sdk` step; `@databricks/sdk-experimental` is a regular npm dependency installed by `yarn install`.

*Verification:*
* Confirmed no `install:sdk` script remains in any package.json and that `@databricks/sdk-experimental` is a dependency in packages/databricks-vscode/package.json.
* Confirmed `packageManager` is `yarn@3.2.1`.

Co-authored-by: Isaac
@github-actions

Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/vscode

Inputs:

  • PR number: 1911
  • Commit SHA: d62853f031be3418d747be20574344518ae72255

Checks will be approved automatically on success.

@rugpanov rugpanov merged commit 260248e into main Jun 16, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants