Skip to content

Conversation

@roulpriya
Copy link
Contributor

@roulpriya roulpriya commented Oct 7, 2025

Description

This PR adds the ability for the Swift extension to install the swiftly toolchain manager if it is not already present on the user's machine. If a .swift-version file is present, but swiftly is not detected, a warning message will be shown to the user telling them that they can use swiftly to manage their toolchains.

The Swift extension will download swiftly, install it, and initialize it with the default environment. It will then install all of the toolchains that it found in the .swift-version file(s). Unfortunately, we also need to ask the user to restart VS Code in order to pick up environment changes.

Issue: #2025

Tasks

  • Required tests have been written
  • [ ] Documentation has been updated
  • Added an entry to CHANGELOG.md if applicable

@roulpriya roulpriya force-pushed the swiftly-install-via-vscode branch from 7905f88 to bbbe271 Compare October 7, 2025 17:56
@roulpriya roulpriya force-pushed the swiftly-install-via-vscode branch from bbbe271 to d6cb6ce Compare October 30, 2025 04:14
@roulpriya roulpriya marked this pull request as ready for review October 30, 2025 04:15
@roulpriya roulpriya force-pushed the swiftly-install-via-vscode branch from d6cb6ce to 582b344 Compare November 15, 2025 06:38
Copy link
Member

@matthewbastien matthewbastien left a comment

Choose a reason for hiding this comment

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

Looking good so far, thank you! I have several comments from my first look at this.

You'll also have to rebase on top of main before adding to the changelog.

@roulpriya roulpriya force-pushed the swiftly-install-via-vscode branch 2 times, most recently from bd9fd51 to 1c8cb8c Compare January 10, 2026 13:11
@roulpriya roulpriya force-pushed the swiftly-install-via-vscode branch 4 times, most recently from 8ae2113 to f485f66 Compare January 16, 2026 15:44
@matthewbastien matthewbastien force-pushed the swiftly-install-via-vscode branch 2 times, most recently from 418d8ce to e13fe0b Compare January 21, 2026 20:07
@matthewbastien matthewbastien force-pushed the swiftly-install-via-vscode branch from e13fe0b to b84edf6 Compare January 21, 2026 20:51
@matthewbastien matthewbastien force-pushed the swiftly-install-via-vscode branch from effd490 to 2ec50b2 Compare January 23, 2026 15:56
@matthewbastien matthewbastien self-requested a review January 23, 2026 17:44
@matthewbastien matthewbastien dismissed their stale review January 23, 2026 17:45

Dismissing my own review as I submitted changes to this PR myself. Need someone else to review.

@matthewbastien matthewbastien removed their request for review January 23, 2026 17:46
@matthewbastien matthewbastien merged commit ea13898 into swiftlang:main Jan 23, 2026
39 of 40 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.

3 participants