Skip to content

Commit f28db1a

Browse files
ANcpLuaclaude
andcommitted
fix: correct DiagnosticClasses polyfill path, simplify test infrastructure
Fix shared source path for DiagnosticClasses.cs and Lock.cs polyfills. Remove redundant test infrastructure and consolidate test files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 7146414 commit f28db1a

10 files changed

Lines changed: 9 additions & 789 deletions

File tree

src/Build/Common/Common.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@
358358
/>
359359
</Target>
360360

361-
<!-- Disable packages -->
361+
<!-- Disable packages (Rider XSD false positive: Remove-only Analyzer items are valid MSBuild) -->
362362
<Target Name="Disable_SponsorLink" BeforeTargets="CoreCompile" Condition="'$(Disable_SponsorLink)' != 'false'">
363363
<ItemGroup>
364364
<Analyzer Remove="@(Analyzer)" Condition="'%(Analyzer.Filename)' == 'DevLooped.SponsorLink'"/>

src/Build/Common/LegacySupport.targets

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@
178178
StackTraceHiddenAttribute
179179
───────────────────────────────────────────────────────────────────────── -->
180180
<ItemGroup Condition="'$(InjectDiagnosticClassesOnLegacy)' == 'true' AND ('$(TargetFramework)' == 'netstandard2.0' OR '$(TargetFramework)' == 'net472')">
181-
<Compile Include="$(_SharedSourcePath)Polyfills/DiagnosticClasses.cs" LinkBase="Shared/Polyfills" InProject="false"/>
181+
<Compile Include="$(_SharedSourcePath)MSBuild/Polyfills/DiagnosticClasses.cs" LinkBase="Shared/Polyfills" InProject="false"/>
182182
</ItemGroup>
183183

184184
<ItemGroup Condition="('$(InjectStackTraceHiddenOnLegacy)' == 'true' OR '$(InjectANcpLuaDiagnosticsPolyfills)' == 'true') AND '$(_NeedsNet6Polyfills)' == 'true'">
@@ -242,7 +242,7 @@
242242
POLYFILLS - Lock & TimeProvider
243243
───────────────────────────────────────────────────────────────────────── -->
244244
<ItemGroup Condition="'$(InjectLockPolyfill)' == 'true' AND '$(_NeedsNet9Polyfills)' == 'true'">
245-
<Compile Include="$(_SharedSourcePath)Polyfills/Lock.cs" LinkBase="Shared/Polyfills" InProject="false"/>
245+
<Compile Include="$(_SharedSourcePath)MSBuild/Polyfills/Lock.cs" LinkBase="Shared/Polyfills" InProject="false"/>
246246
</ItemGroup>
247247

248248
<!-- ─────────────────────────────────────────────────────────────────────────

tests/ANcpLua.Sdk.Tests/ClaudeBrainTests.cs

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

tests/ANcpLua.Sdk.Tests/Helpers/SdkProjectBuilder.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ public void AddDirectoryBuildPropsFile(string postSdkContent, string preSdkConte
343343
</Project>
344344
""";
345345

346-
AddFile(RepositoryPaths.DirectoryBuildProps, content);
346+
AddFile("Directory.Build.props", content);
347347
}
348348

349349
/// <inheritdoc />

tests/ANcpLua.Sdk.Tests/Infrastructure/PolyfillProps.cs

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

0 commit comments

Comments
 (0)