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

Modify VSCode extension to use dotnet tool list + only warn to instal… #1323

Merged
merged 2 commits into from
Aug 18, 2024

Conversation

belav
Copy link
Owner

@belav belav commented Aug 18, 2024

This modifies the VSCode extension in the following ways

  • Instead of looking for dotnet-tools.json files in the current directory and parent directories the extension runs dotnet tool list and parses the outpud looking for the csharpier version.
  • Instead of running dotnet csharpier --version to look for the global version it repeats the above step with dotnet tool list -g
  • The "You need to install csharpier" window was popping up every time you opened a file if it couldn't find an install. The other extensions were limited to showing the warning a single time. VSCode now only shows it a single time.
  • Files under /DecompilationMetadataAsSourceFileProvider/ will never have the warning show. This is where the c# dev kit decompiles code.

#1267

@belav belav merged commit 2456512 into main Aug 18, 2024
3 checks passed
@belav belav deleted the use-dotnet-output-vscode branch August 18, 2024 18:27
pisolofin pushed a commit to pisolofin/csharpier-editorconfig that referenced this pull request Aug 22, 2024
belav#1323)

This modifies the VSCode extension in the following ways
- Instead of looking for dotnet-tools.json files in the current
directory and parent directories the extension runs `dotnet tool list`
and parses the outpud looking for the csharpier version.
- Instead of running `dotnet csharpier --version` to look for the global
version it repeats the above step with `dotnet tool list -g`
- The "You need to install csharpier" window was popping up every time
you opened a file if it couldn't find an install. The other extensions
were limited to showing the warning a single time. VSCode now only shows
it a single time.
- Files under `/DecompilationMetadataAsSourceFileProvider/` will never
have the warning show. This is where the c# dev kit decompiles code.

belav#1267
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.

1 participant