Skip to content

Commit 2fc18e5

Browse files
authored
Include analyzers in main NuGet package (#1)
- Add ProjectReference to Weaviate.Client.Managed.Analyzers - Package analyzer DLL in analyzers/dotnet/cs path - Remove separate analyzer packaging from workflow - Users now get compile-time validation automatically when installing Weaviate.Client.Managed
1 parent 2ce7c2a commit 2fc18e5

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

.github/workflows/main.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,6 @@ jobs:
270270
- name: Create NuGet package
271271
run: |
272272
dotnet pack -c Release -o out src/Weaviate.Client.Managed/
273-
dotnet pack -c Release -o out src/Weaviate.Client.Managed.Analyzers/
274273
275274
- name: Push package to NuGet
276275
run: dotnet nuget push './out/*.nupkg' --skip-duplicate --api-key ${{ secrets.NUGET_APIKEY }} --source https://api.nuget.org/v3/index.json

src/Weaviate.Client.Managed/Weaviate.Client.Managed.csproj

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,21 @@
7979
<PackageReference Include="Weaviate.Client" Version="1.0.1" />
8080
</ItemGroup>
8181

82+
<!-- Include Roslyn analyzers in the package -->
83+
<ItemGroup>
84+
<ProjectReference Include="..\Weaviate.Client.Managed.Analyzers\Weaviate.Client.Managed.Analyzers.csproj"
85+
ReferenceOutputAssembly="false"
86+
PrivateAssets="all" />
87+
</ItemGroup>
88+
89+
<!-- Package the analyzer DLL -->
90+
<ItemGroup>
91+
<None Include="..\Weaviate.Client.Managed.Analyzers\bin\$(Configuration)\netstandard2.0\Weaviate.Client.Managed.Analyzers.dll"
92+
Pack="true"
93+
PackagePath="analyzers/dotnet/cs"
94+
Visible="false" />
95+
</ItemGroup>
96+
8297
<ItemGroup>
8398
<None Include="README.md" Pack="true" PackagePath="\" />
8499
<None Include="..\..\icon.png" Pack="true" PackagePath="\" Condition="Exists('..\..\icon.png')" />

0 commit comments

Comments
 (0)