Skip to content

update dependency deployment#60469

Merged
jmarolf merged 2 commits intodotnet:mainfrom
jmarolf:bugfix/update-dependencies-in-telemetry-generator-tools
Mar 30, 2022
Merged

update dependency deployment#60469
jmarolf merged 2 commits intodotnet:mainfrom
jmarolf:bugfix/update-dependencies-in-telemetry-generator-tools

Conversation

@jmarolf
Copy link
Copy Markdown
Contributor

@jmarolf jmarolf commented Mar 29, 2022

There are several dependencies that are set to PrivateAssets="all" in Microsoft.CodeAnalysis.EditorFeatures here:

<!-- The ImageCatalog reference is only used at compile time, not at runtime, because it is not supported on Mac.
There is a test to ensure this doesn't regress in src\EditorFeatures\Test\AssemblyReferenceTests.cs -->
<PackageReference Include="Microsoft.VisualStudio.ImageCatalog" Version="$(MicrosoftVisualStudioImageCatalogVersion)" PrivateAssets="all" />
<PackageReference Include="Microsoft.VisualStudio.Language" Version="$(MicrosoftVisualStudioLanguageVersion)" />
<!-- No warn on NU1701 until the LSP client targets netstandard2.0 https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1369985/ -->
<PackageReference Include="Microsoft.VisualStudio.LanguageServer.Client" Version="$(MicrosoftVisualStudioLanguageServerClientVersion)" PrivateAssets="all" NoWarn="NU1701" />
<!-- Explicit reference to Shell.15.0 et. al. with NU1701 only until the LSP client targets netstandard2.0 https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1369985/ -->
<PackageReference Include="Microsoft.VisualStudio.Shell.15.0" Version="$(MicrosoftVisualStudioShell150Version)" PrivateAssets="all" ExcludeAssets="all" NoWarn="NU1701" />
<PackageReference Include="Microsoft.VisualStudio.Shell.Framework" Version="$(MicrosoftVisualStudioShellFrameworkVersion)" PrivateAssets="all" ExcludeAssets="all" NoWarn="NU1701" />
<PackageReference Include="Microsoft.VisualStudio.GraphModel" Version="$(MicrosoftVisualStudioGraphModelVersion)" PrivateAssets="all" ExcludeAssets="all" NoWarn="NU1701" />
<PackageReference Include="Microsoft.VisualStudio.Imaging" Version="$(MicrosoftVisualStudioImagingVersion)" PrivateAssets="all" ExcludeAssets="all" NoWarn="NU1701" />

We need to deploy these so we can properly reflect over Microsoft.CodeAnalysis.EditorFeatures and discover the set of codefixes and refactorings we ship and compute the telemetry hashes for them.

@jmarolf jmarolf requested a review from a team as a code owner March 29, 2022 23:13
@ghost ghost added the Area-Infrastructure label Mar 29, 2022
@jmarolf jmarolf requested a review from JoeRobich March 29, 2022 23:13
<PropertyGroup>
<OutputType>Exe</OutputType>
<TargetFrameworks>net6.0</TargetFrameworks>
<TargetFrameworks>net6.0-windows</TargetFrameworks>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because this reference does not target netstandard or net6.0 I needed to change the tfm to be windows only for now

@JoeRobich
Copy link
Copy Markdown
Member

Do you also need to update the CodeActionDescriptionMap in the Program.cs?

@jmarolf jmarolf enabled auto-merge March 29, 2022 23:28
@jmarolf jmarolf merged commit 998e65d into dotnet:main Mar 30, 2022
@ghost ghost added this to the Next milestone Mar 30, 2022
@dibarbet dibarbet modified the milestones: Next, 17.3.P1 Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants