Skip to content

Commit c7de65f

Browse files
Fix docs file names and add validation (#107359)
Co-authored-by: Eric StJohn <[email protected]>
1 parent 810c806 commit c7de65f

File tree

2 files changed

+22
-1
lines changed

2 files changed

+22
-1
lines changed

eng/intellisense.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
<IntellisensePackageXmlFilePath Condition="'$(IntellisensePackageXmlFilePath)' == '' and Exists($(IntellisensePackageXmlFilePathFromNetFolder))">$(IntellisensePackageXmlFilePathFromNetFolder)</IntellisensePackageXmlFilePath>
1212
<IntellisensePackageXmlFilePath Condition="'$(IntellisensePackageXmlFilePath)' == '' and Exists($(IntellisensePackageXmlFilePathFromDotNetPlatExtFolder))">$(IntellisensePackageXmlFilePathFromDotNetPlatExtFolder)</IntellisensePackageXmlFilePath>
1313

14-
<IntermediateDocFileItemFromIntellisensePackage>$(IntermediateOutputPath)$(TargetName).intellisense-package.xml</IntermediateDocFileItemFromIntellisensePackage>
14+
<IntermediateDocFileItemFromIntellisensePackage>$([MSBuild]::NormalizePath('$(IntermediateOutputPath)', 'intellisense-package', '$(TargetName).xml'))</IntermediateDocFileItemFromIntellisensePackage>
1515

1616
<!-- Suppress "CS1591 - Missing XML comment for publicly visible type or member" compiler errors when the intellisense package xml file is used. -->
1717
<NoWarn Condition="'$(SkipIntellisenseNoWarnCS1591)' != 'true'">$(NoWarn);CS1591</NoWarn>

src/libraries/sfx.proj

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,27 @@
7373
UseHardlinksIfPossible="true" />
7474
</Target>
7575

76+
<Target Name="ValidateSharedFramework"
77+
AfterTargets="Build">
78+
<ItemGroup>
79+
<!-- exclude private assemblies from ref-->
80+
<_expectedRef Include="@(NetCoreAppLibrary)" Condition="!$([System.String]::new('%(Identity)').StartsWith('System.Private'))" />
81+
<_expectedSharedFrameworkFile Include="@(_expectedRefs->'$(MicrosoftNetCoreAppRefPackRefDir)%(Identity).dll')" />
82+
83+
<!-- exclude the full facades from expected docs since they have no types -->
84+
<_expectedDoc Include="@(_expectedRefs)" Exclude="netstandard;@(NetFxReference)" />
85+
<_expectedSharedFrameworkFile Include="@(_expectedDoc->'$(MicrosoftNetCoreAppRefPackRefDir)%(Identity).xml')" />
86+
87+
<!-- exclude CoreLib from expected libs, since it's placed in native -->
88+
<_expectedLib Include="@(NetCoreAppLibrary)" Exclude="System.Private.CoreLib" />
89+
<_expectedSharedFrameworkFile Include="@(_expectedLib->'$(MicrosoftNetCoreAppRuntimePackRidLibTfmDir)%(Identity).dll')" />
90+
91+
<_missingSharedFrameworkFile Include="@(_expectedSharedFrameworkFile)" Exclude="@(_expectedSharedFrameworkFile->Exists())" />
92+
</ItemGroup>
93+
94+
<Error Text="The shared framework files '@(_missingSharedFrameworkFile)' were missing." Condition="'@(_missingSharedFrameworkFile)' != ''" />
95+
</Target>
96+
7697
<Target Name="GetTrimSharedFrameworkAssembliesInputs"
7798
DependsOnTargets="ResolveProjectReferences">
7899
<PropertyGroup>

0 commit comments

Comments
 (0)