Skip to content

Commit eff7d84

Browse files
committed
Making restore test targets no-ops temporarily
1 parent d677cb1 commit eff7d84

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

build.proj

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@
8888
<Target Name="BuildAllConfigurations" DependsOnTargets="Restore;BuildTools;BuildSqlServerLib;BuildNetFx;BuildNetCoreAllOS;BuildNetStandard;GenerateMdsPackage" />
8989
<Target Name="BuildSqlServerPackage" DependsOnTargets="BuildSqlServerLibAnyOS;GenerateSqlServerPackage"/>
9090

91-
<!-- @TODO: Manual tests have a reference to AKV provider, and AKV provider is not built on non-windows pipeline runs. Thus it is added to the dependencies of the build tests target. This is not good since it means AKV is built twice on windows and once on unix, in a mix of implicit and explicit. The pipeline should be modified to build it in both cases, or the build should be updated to have dotnet build the test projects. -->
9291
<Target Name="BuildTestsNetCore">
9392
<Message Text=">>> No op. Will be removed." />
9493
</Target>
@@ -105,16 +104,16 @@
105104
<MSBuild Projects="@(NetCoreDriver)" Targets="restore" Properties="$(ProjectProperties)" />
106105
</Target>
107106

108-
<Target Name="RestoreTestsNetCore" DependsOnTargets="RestoreNetCore">
109-
<MSBuild Projects="@(ManualTests)" Targets="restore" Properties="$(TestProjectProperties)" />
107+
<Target Name="RestoreTestsNetCore">
108+
<Message Text=">>> No op. Will be removed." />
110109
</Target>
111110

112111
<Target Name="RestoreNetFx" DependsOnTargets="RestoreSqlServerLib" Condition="'$(IsEnabledWindows)' == 'true'">
113112
<MSBuild Projects="@(NetFxDriver)" Targets="restore" Properties="$(ProjectProperties)" />
114113
</Target>
115114

116-
<Target Name="RestoreTestsNetFx" DependsOnTargets="RestoreNetFx" Condition="'$(IsEnabledWindows)' == 'true'">
117-
<MSBuild Projects="@(ManualTests)" Targets="restore" Properties="$(TestProjectProperties)" />
115+
<Target Name="RestoreTestsNetFx" Condition="'$(IsEnabledWindows)' == 'true'">
116+
<Message Text=">>> No op. Will be removed." />
118117
</Target>
119118

120119
<Target Name="RestoreTools" Condition="'$(BuildTools)' == 'true'">

0 commit comments

Comments
 (0)