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

Create a transport package for VS and VS authoring for analyzers #42451

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

marcpopMSFT
Copy link
Member

#42087

  • Create a new nupkg to package up the analyzers and VS authoring
  • Copy the analyzers into a separate folder to be included in the nupkg
  • Create separate sub-folders for each category of analyzer pulling the contents from the analyzers folder
  • Install into VS into the SDK<analyzercategory> folder
  • Mark all the files to be ngened
  • Keep the existing folder structure

Some examples:
folder "InstallDir:\SDK\AspNetCoreAnalyzers\dotnet\cs"
file source="$(PkgVS_Redist_Common_Net_Core_SDK_RuntimeAnalyzers)\AspNetCoreAnalyzers\9.0.0-preview.5.24306.11\analyzers\dotnet\cs\Microsoft.AspNetCore.App.Analyzers.dll" vs.file.ngenApplications="[installDir]\Common7\IDE\vsn.exe"

folder "InstallDir:\SDK\NetCoreAnalyzers\dotnet\cs"
file source="$(PkgVS_Redist_Common_Net_Core_SDK_RuntimeAnalyzers)\NetCoreAnalyzers\9.0.0-preview.5.24306.7\analyzers\dotnet\cs\System.Text.RegularExpressions.Generator.dll" vs.file.ngenApplications="[installDir]\Common7\IDE\vsn.exe"

folder "InstallDir:\SDK\WindowsDesktopAnalyzers\dotnet"
file source="$(PkgVS_Redist_Common_Net_Core_SDK_RuntimeAnalyzers)\WindowsDesktopAnalyzers\9.0.0-preview.5.24306.8\analyzers\dotnet\System.Windows.Forms.Analyzers.dll" vs.file.ngenApplications="[installDir]\Common7\IDE\vsn.exe"

folder "InstallDir:\SDK\SDKAnalyzers"
file source="$(PkgVS_Redist_Common_Net_Core_SDK_RuntimeAnalyzers)\SDKAnalyzers\9.0.100-dev\Sdks\Microsoft.NET.Sdk\analyzers\Microsoft.CodeAnalysis.NetAnalyzers.dll" vs.file.ngenApplications="[installDir]\Common7\IDE\vsn.exe"

folder "InstallDir:\SDK\WebSDKAnalyzers\cs"
file source="$(PkgVS_Redist_Common_Net_Core_SDK_RuntimeAnalyzers)\WebSDKAnalyzers\9.0.100-dev\Sdks\Microsoft.NET.Sdk.Web\analyzers\cs\Microsoft.AspNetCore.Analyzers.dll" vs.file.ngenApplications="[installDir]\Common7\IDE\vsn.exe"

There are 5 analyzers we want to include.
Other teams are investigating a potentially mapping and final location.
This is just a placeholder to generate the package.
This is also WIP as I don't have the VS side of things finding the right files yet.
Put the analyzers in the SDK folder in a separate sub-folder for each.
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Jul 29, 2024

AddFolder(sb,
@"AspNetCoreAnalyzers",
@"SDK\AspNetCoreAnalyzers");
Copy link
Member

Choose a reason for hiding this comment

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

Should the target VS folder be something like DotNetSdk rather than just SDK?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants