Skip to content

Commit

Permalink
fix: correct Condition for stylecop.json inclusion
Browse files Browse the repository at this point in the history
Condition for `stylecop.json` inclusion in `Allegro.NET.Sdk.CSharp.targets` had an extra `)` at the end after previous PR removed part of condition, which resulted in MSBuild parsing errors.
  • Loading branch information
amis92 authored Apr 9, 2024
1 parent 1766ef1 commit fcd3696
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.targets
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,9 @@
</PropertyGroup>

<!--
Include stylecop configuration if we include StyleCop analyzer within this Sdk,
or if it was included externally as GlobalPackageReference when CPM is enabled.
Include stylecop configuration if StyleCop was included externally as GlobalPackageReference when CPM is enabled.
-->
<ItemGroup Condition="'$(AllegroDotnetSdkEnableStyleCopConfig)' == 'true' Or @(GlobalPackageReference->WithMetadataValue('Identity','StyleCop.Analyzers')->Count()) > 0)">
<ItemGroup Condition="'$(AllegroDotnetSdkEnableStyleCopConfig)' == 'true' And @(GlobalPackageReference->WithMetadataValue('Identity','StyleCop.Analyzers')->Count()) > 0">
<AdditionalFiles Include="$(MSBuildThisFileDirectory)stylecop.json" Link="stylecop.json" />
</ItemGroup>

Expand All @@ -23,4 +22,4 @@
<Using Remove="System.Net.Http" />
</ItemGroup>

</Project>
</Project>

0 comments on commit fcd3696

Please sign in to comment.