Skip to content

Commit a920a84

Browse files
Remove --no-build flag and clean up project dependencies
Removed the `--no-build` flag from `dotnet pack` commands in `deterministic-build.yml` to ensure the project is built during packaging. This affects both the initial package creation and verification steps. In `Codebreaker.Analyzers.csproj`, removed the `<WarningsAsErrors />` element, allowing warnings to no longer be treated as errors. Also removed the `<PackageReference>` to `System.Text.Json`, as this was a workaround for the project or NuGet packaging.
1 parent cb2b80c commit a920a84

2 files changed

Lines changed: 0 additions & 8 deletions

File tree

.github/workflows/deterministic-build.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ jobs:
8989
echo "Creating deterministic NuGet package for ${{ matrix.solution.name }}..."
9090
dotnet pack ${{ matrix.solution.project }} \
9191
--configuration Release \
92-
--no-build \
9392
--output ./packages \
9493
/p:ContinuousIntegrationBuild=true \
9594
/p:Deterministic=true \
@@ -110,7 +109,6 @@ jobs:
110109
echo "Building second time for verification..."
111110
dotnet pack ${{ matrix.solution.project }} \
112111
--configuration Release \
113-
--no-build \
114112
--output ./packages-verify \
115113
/p:ContinuousIntegrationBuild=true \
116114
/p:Deterministic=true \

src/services/common/Codebreaker.GameAPIs.Analyzers/Codebreaker.Analyzers.csproj

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,11 @@
2020
<!-- Suppress NU5017 warning for packages with no dependencies -->
2121
<NoWarn>$(NoWarn);NU5017</NoWarn>
2222
<MSBuildTreatWarningsAsErrors>false</MSBuildTreatWarningsAsErrors>
23-
<WarningsAsErrors />
2423
</PropertyGroup>
2524

2625
<ItemGroup>
2726
<None Include="docs/readme.md" Pack="true" PackagePath="\" />
2827
<None Include="Images/codebreaker.jpeg" Pack="true" PackagePath="\" />
2928
</ItemGroup>
3029

31-
<ItemGroup>
32-
<!-- Add minimal dependency to satisfy NuGet packaging requirements -->
33-
<PackageReference Include="System.Text.Json" />
34-
</ItemGroup>
35-
3630
</Project>

0 commit comments

Comments
 (0)