Remove Visual Studio Code and IDE extensions - #790
Open
Giulia Stocco (gfs) wants to merge 7 commits into
Open
Giulia Stocco (gfs) wants to merge 7 commits into
Giulia Stocco (gfs) wants to merge 7 commits into
Conversation
…ode and Visual Studio extensions, including version management and package handling.
Delete Microsoft.DevSkim.VisualStudio, Microsoft.DevSkim.LanguageServer, and Microsoft.DevSkim.LanguageProtoInterop, remove them from the solution, and drop the Visual Studio-only media assets.
Delete the root .vscode extension-development config, drop the npm ecosystem from Dependabot, and remove extension build artifacts from .gitignore.
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The solution now has an invalid leading blank line, and the documented SDK minimum cannot build its .NET 10 targets.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 1
Open (2)
What changed in this PR
Removes deprecated IDE extensions and refocuses DevSkim on its .NET library and CLI.
Changes:
- Deletes Visual Studio and VS Code extension code, supporting language-server projects, and pipelines.
- Updates repository configuration and documentation.
- Adds a changelog entry.
| File | Description |
|---|---|
README.md |
Refocuses documentation on the library and CLI. |
Pipelines/vscode/devskim-vscode-release.yml |
Removes VS Code release pipeline. |
Pipelines/vscode/devskim-vscode-pr.yml |
Removes VS Code PR pipeline. |
Pipelines/vs/devskim-visualstudio-release.yml |
Removes Visual Studio release pipeline. |
Pipelines/vs/devskim-visualstudio-pr.yml |
Removes Visual Studio PR pipeline. |
DevSkim-VSCode-Plugin/tsconfig.json |
Removes extension TypeScript configuration. |
DevSkim-VSCode-Plugin/ThirdPartyNotices.txt |
Removes extension notices. |
DevSkim-VSCode-Plugin/scripts/updatePackageLock.js |
Removes package-lock helper. |
DevSkim-VSCode-Plugin/scripts/setPackageVersion.js |
Removes versioning helper. |
DevSkim-VSCode-Plugin/scripts/resetPackageVersion.js |
Removes version reset helper. |
DevSkim-VSCode-Plugin/scripts/checkPreReleaseFlag.js |
Removes prerelease helper. |
DevSkim-VSCode-Plugin/README.md |
Removes extension documentation. |
DevSkim-VSCode-Plugin/package.json |
Removes extension package definition. |
DevSkim-VSCode-Plugin/License.txt |
Removes extension license copy. |
DevSkim-VSCode-Plugin/eslint.config.mjs |
Removes extension lint configuration. |
DevSkim-VSCode-Plugin/client/tsconfig.json |
Removes client compiler configuration. |
DevSkim-VSCode-Plugin/client/package.json |
Removes client package definition. |
DevSkim-VSCode-Plugin/client/package-lock.json |
Removes client dependency lock. |
DevSkim-VSCode-Plugin/client/extension.ts |
Removes VS Code entry point. |
DevSkim-VSCode-Plugin/client/devSkimFixer.ts |
Removes VS Code quick fixes. |
DevSkim-VSCode-Plugin/client/common/selectors.ts |
Removes language selectors. |
DevSkim-VSCode-Plugin/client/common/notificationNames.ts |
Removes LSP notification names. |
DevSkim-VSCode-Plugin/client/common/fileVersion.ts |
Removes file-version model. |
DevSkim-VSCode-Plugin/client/common/devskimSettings.ts |
Removes extension settings model. |
DevSkim-VSCode-Plugin/client/common/codeFixMapping.ts |
Removes client fix mapping. |
DevSkim-VSCode-Plugin/.vscodeignore |
Removes extension packaging exclusions. |
DevSkim-VSCode-Plugin/.vscode/tasks.json |
Removes extension tasks. |
DevSkim-VSCode-Plugin/.vscode/settings.json |
Removes extension workspace settings. |
DevSkim-VSCode-Plugin/.vscode/launch.json |
Removes extension launch profiles. |
DevSkim-VSCode-Plugin/.vscode/extensions.json |
Removes extension recommendations. |
DevSkim-VSCode-Plugin/.npmrc.pipeline |
Removes npm feed configuration. |
DevSkim-VSCode-Plugin/.gitignore |
Removes extension-specific ignores. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/publish.manifest.json |
Removes marketplace manifest. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/Properties/launchSettings.json |
Removes Visual Studio launch profile. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/Microsoft.DevSkim.VisualStudio.csproj |
Removes Visual Studio project. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/DevSkimSettingDefinitions.cs |
Removes extension settings definitions. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/DevSkimLanguageServerProvider.cs |
Removes language-server integration. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/DevSkimExtension.cs |
Removes extension entry point. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/Content/readme-gallery.md |
Removes marketplace documentation. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/Content/License.txt |
Removes packaged license copy. |
DevSkim-DotNet/Microsoft.DevSkim.VisualStudio/.vsextension/string-resources.json |
Removes localized extension resources. |
DevSkim-DotNet/Microsoft.DevSkim.sln |
Removes IDE-related projects from the solution. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/VisualStudioConfigurationHandler.cs |
Removes Visual Studio configuration handler. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/TextDocumentSyncHandler.cs |
Removes LSP document synchronization. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/StaticScannerSettings.cs |
Removes language-server settings. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/Program.cs |
Removes language-server entry point. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/Microsoft.DevSkim.LanguageServer.csproj |
Removes language-server project. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/DidChangeConfigurationHandler.cs |
Removes configuration-change handler. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/ConfigHelpers.cs |
Removes configuration helpers. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageServer/CodeActionHandler.cs |
Removes LSP code actions. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/ValueResultEnum.cs |
Removes settings result enum. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/PortableScannerSettings.cs |
Removes portable settings model. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/Microsoft.DevSkim.LanguageProtoInterop.csproj |
Removes protocol interoperability project. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/Messages.cs |
Removes protocol message constants. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/IDevSkimOptions.cs |
Removes shared IDE options interface. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/CommentStylesEnum.cs |
Removes comment-style enum. |
DevSkim-DotNet/Microsoft.DevSkim.LanguageProtoInterop/CodeFixMapping.cs |
Removes server fix mapping. |
Changelog.md |
Records IDE extension removal. |
.vscode/tasks.json |
Removes root extension build tasks. |
.vscode/settings.json |
Removes root TypeScript settings. |
.vscode/launch.json |
Removes root extension launch profiles. |
.vscode/extensions.json |
Removes root extension recommendations. |
.gitignore |
Removes obsolete extension artifacts. |
.github/dependabot.yml |
Stops npm dependency monitoring. |
.github/copilot-instructions.md |
Removes IDE-extension development guidance. |
Files not reviewed (1)
- DevSkim-VSCode-Plugin/client/package-lock.json: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Updated .NET SDK version requirement in README. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Eliminate deprecated scripts, configurations, and projects related to the DevSkim Visual Studio Code and Visual Studio extensions. Update documentation to reflect the focus on the library and CLI. Add a changelog entry for the removal of IDE extensions.