Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing tunnel module that is stopping VSIX activation #18712

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

lewis-sanchez
Copy link
Contributor

@lewis-sanchez lewis-sanchez commented Feb 19, 2025

This PR is for this issue: #18713

This PR fixes the following issue that is preventing the VSCode for MSSQL extension from starting up.

image

I verified that the change resolves this error by installing the VSIX from the buddy build drop here: https://msdata.visualstudio.com/SQLToolsAndLibraries/_build/results?buildId=160997762&view=artifacts&pathAsName=false&type=publishedArtifacts

Extension host output for installing a VSIX with the fix from the buddy build artifact drop:
image

@cssuh
Copy link
Member

cssuh commented Feb 19, 2025

Could you create an issue and link it here? especially if this is something for the feb release

Copy link

PR Changes

Category Main Branch PR Branch Difference
Code Coverage 50.40% 50.54% $${\color{lightgreen} .14\% }$$
VSIX Size 12958 KB 12969 KB $${\color{lightgreen} 11 KB \space (0\%) }$$
Webview Bundle Size 3188 KB 3188 KB $${\color{lightgreen} 0 KB \space (0\%) }$$

@lewis-sanchez
Copy link
Contributor Author

Could you create an issue and link it here? especially if this is something for the feb release

Updated PR description with the issue.

@lewis-sanchez lewis-sanchez merged commit 2847fdc into main Feb 19, 2025
6 checks passed
@lewis-sanchez lewis-sanchez deleted the lewissanchez/fix/tunnel-module-not-found branch February 19, 2025 22:30
lewis-sanchez added a commit that referenced this pull request Feb 19, 2025
@caohai
Copy link
Member

caohai commented Feb 19, 2025

Do you know if this is a regression? What we use this pkg for? I don't see we ever included this pkg before. @lewis-sanchez

@caohai
Copy link
Member

caohai commented Feb 19, 2025

Ah I see, the usage of tunnel was added via #18679. A few questions:

  1. I don't see this pkg in ADS, is ADS using something different?
  2. This source repo of the pkg is in archive mode, do we have an altertive other than this pkg? It's okay if not.

Benjin pushed a commit that referenced this pull request Feb 20, 2025
kburtram pushed a commit that referenced this pull request Feb 20, 2025
)

* Add missing tunnel module (#18712) (#18714)

* Adding VS Code walkthroughs (#18694) (#18709)

* adding walkthroughs

* formatted

* adding query sort/filter

* loc update

* updated pictures for Getting Started walkthrough

* Adding images for Next Steps

* loc

* fixing json formatting

* fixing merge

* fixing whitespace

* whitespace

* reverting more whitespace

* whitespace

* nls whitespace

* swapping for const

* Fixing typo in package.json and adding missing string in nls.json

* loc

* Scaling screenshots down

* fixing casing

* xlf update

* Added accessibility functionality for query plans (#18711)

* added accesibility labels

* localization

---------

Co-authored-by: Lewis Sanchez <87730006+lewis-sanchez@users.noreply.github.com>
Co-authored-by: Benjin Dubishar <benjin.dubishar@gmail.com>
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.

4 participants