Skip to content

Conversation

@jeremymeng
Copy link
Member

to only include folders that are mostly maintained by the central JS SDK team.
To have all SDKs available in VS Code, open the repository directory instead.

The tssdk path is fixed as core-http has been removed.

Also add "streetsidesoftware.code-spell-checker" to VS Code recommended extensions

to only include folders that are mostly maintained by the central JS SDK team.
To have all SDKs available in VS Code, open the repository directory instead.

The tssdk path is fixed as core-http has been removed.

Also add "streetsidesoftware.code-spell-checker" to VS Code recommended extensions
Copilot AI review requested due to automatic review settings December 19, 2025 22:37
@jeremymeng
Copy link
Member Author

Not sure whether other SDK authors are also using this workspace. We could also rename or have a separate core workspace?

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR streamlines the dataplane.code-workspace configuration to focus on packages maintained by the central JS SDK team. The workspace is reduced from ~50 folders to ~30 folders, removing non-core packages like service-specific SDKs (e.g., communication, storage, cosmos). The PR also fixes an outdated TypeScript SDK path reference and adds the code spell checker extension to the recommended extensions list.

Key Changes

  • Removed service-specific SDK folders (communication, storage, synapse, etc.) to focus on core infrastructure packages
  • Fixed the typescript.tsdk path from the removed core-http to sdk/core/core-rest-pipeline
  • Added trailing commas throughout the JSON for better git diffs and consistency

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
dataplane.code-workspace Reduced workspace folders to core team-maintained packages (~30 folders), fixed TypeScript SDK path, added spell checker extension, and applied consistent trailing commas
.vscode/extensions.json Added spell checker to the recommended extensions list for the repository

@xirzec
Copy link
Member

xirzec commented Dec 19, 2025

TBH I haven't used the workspaces in some time as VS Code became fast enough at opening the root of the repo and we don't have the root package.json poisoning things anymore

@jeremymeng
Copy link
Member Author

TBH I haven't used the workspaces in some time as VS Code became fast enough at opening the root of the repo and we don't have the root package.json poisoning things anymore

Yeah I always open the root directory. Maybe this file can be deleted then.

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