Skip to content

Commit ad0c47c

Browse files
authored
Fix docs file names and add validation (#107334)
1 parent 00632dc commit ad0c47c

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
@@ -69,6 +69,27 @@
6969
UseHardlinksIfPossible="true" />
7070
</Target>
7171

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

0 commit comments

Comments
 (0)