Skip to content

Commit 686ce02

Browse files
authored
Remove code for building an analyzers package that never shipped (#6053)
Microsoft.AspNetCore.Analyzers didn't ship in 2.1, and we don't have plans to every ship it in a 2.1.x release.
1 parent 0cbc52c commit 686ce02

File tree

4 files changed

+3
-38
lines changed

4 files changed

+3
-38
lines changed

build/SharedFx.targets

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
<ItemGroup>
1010
<_MetapackageProject Include="$(RepositoryRoot)src\Packages\Microsoft.AspNetCore.All\Microsoft.AspNetCore.All.csproj" />
1111
<_MetapackageProject Include="$(RepositoryRoot)src\Packages\Microsoft.AspNetCore.App\Microsoft.AspNetCore.App.csproj" />
12-
<_MetapackageProject Include="$(RepositoryRoot)src\Packages\Microsoft.AspNetCore.Analyzers\Microsoft.AspNetCore.Analyzers.csproj" />
1312
</ItemGroup>
1413

1514
<MSBuild Projects="@(_MetapackageProject)"
@@ -92,14 +91,6 @@
9291
MetapackageDependencyVersionRangeType=Minimum
9392
</AdditionalProperties>
9493
</_MetapackageBuilderProject>
95-
96-
<_MetapackageBuilderProject Include="$(MSBuildProjectFullPath)">
97-
<AdditionalProperties>
98-
MetapackageName=Microsoft.AspNetCore.Analyzers;
99-
MetapackageReferenceType=Analyzer;
100-
MetapackageDependencyVersionRangeType=Minimum
101-
</AdditionalProperties>
102-
</_MetapackageBuilderProject>
10394
</ItemGroup>
10495

10596
<MSBuild

build/artifacts.props

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
<AppMetapackage>false</AppMetapackage>
77
<!-- When true, this dependency will be included in the Microsoft.AspNetCore.All metapackage. -->
88
<AllMetapackage>false</AllMetapackage>
9-
<!-- When true, this dependency will be included in the Microsoft.AspNetCore.Analyzers metapackage. -->
10-
<Analyzer>false</Analyzer>
119
<!--
1210
Other known package types:
1311
Dependency = for packages that are installed via PackageReference
@@ -26,7 +24,6 @@
2624
<PackageArtifact Include="dotnet-watch" Category="ship" />
2725
<PackageArtifact Include="Microsoft.AspNet.Identity.AspNetCoreCompat" Category="noship" />
2826
<PackageArtifact Include="Microsoft.AspNetCore.All" Category="ship" />
29-
<PackageArtifact Include="Microsoft.AspNetCore.Analyzers" Category="shipoob" />
3027
<PackageArtifact Include="Microsoft.AspNetCore.Antiforgery" AllMetapackage="true" AppMetapackage="true" Category="ship" />
3128
<PackageArtifact Include="Microsoft.AspNetCore.App" Category="ship" />
3229
<PackageArtifact Include="Microsoft.AspNetCore.ApplicationInsights.HostingStartup" AllMetapackage="true" Category="ship" />
@@ -89,8 +86,7 @@
8986
<PackageArtifact Include="Microsoft.AspNetCore.Localization" AllMetapackage="true" AppMetapackage="true" Category="ship" />
9087
<PackageArtifact Include="Microsoft.AspNetCore.MiddlewareAnalysis" AllMetapackage="true" AppMetapackage="true" Category="ship" />
9188
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Abstractions" AllMetapackage="true" AppMetapackage="true" Category="ship" />
92-
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Analyzers.Experimental" Analyzer="true" Category="shipoob" />
93-
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Analyzers" Analyzer="true" AllMetapackage="true" AppMetapackage="true" Category="ship" />
89+
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Analyzers" AllMetapackage="true" AppMetapackage="true" Category="ship" />
9490
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.ApiExplorer" AllMetapackage="true" AppMetapackage="true" Category="ship" />
9591
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Core" AllMetapackage="true" AppMetapackage="true" Category="ship" />
9692
<PackageArtifact Include="Microsoft.AspNetCore.Mvc.Cors" AllMetapackage="true" AppMetapackage="true" Category="ship" />

src/Mvc/src/Microsoft.AspNetCore.Mvc.Analyzers.Experimental/Microsoft.AspNetCore.Mvc.Analyzers.Experimental.csproj

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,8 @@
22
<PropertyGroup>
33
<Description>CSharp Analyzers for ASP.NET Core MVC.</Description>
44
<PackageTags>aspnetcore;aspnetcoremvc</PackageTags>
5-
6-
<VerifyVersion>false</VerifyVersion>
7-
<VersionPrefix>$(ExperimentalVersionPrefix)</VersionPrefix>
8-
<VersionSuffix>$(ExperimentalVersionSuffix)</VersionSuffix>
9-
<PackageVersion>$(ExperimentalPackageVersion)</PackageVersion>
10-
5+
<!-- These analyzers do not need to put into a package. -->
6+
<IsPackable>false</IsPackable>
117
<TargetFramework>netstandard1.3</TargetFramework>
128
<IncludeBuildOutput>false</IncludeBuildOutput>
139
<EnableApiCheck>false</EnableApiCheck>

src/Packages/Microsoft.AspNetCore.Analyzers/Microsoft.AspNetCore.Analyzers.csproj

Lines changed: 0 additions & 18 deletions
This file was deleted.

0 commit comments

Comments
 (0)