Skip to content

Commit 3032d6c

Browse files
committed
Review comments: Reword warning and error
- explain a bit more about missing .dotnet/ layouts
1 parent fbfbf5f commit 3032d6c

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

eng/helix/helix.proj

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,10 @@
112112
</PropertyGroup>
113113

114114
<!-- After complaining about missing App.Ref layout, do nothing else in this target. -->
115-
<Warning Text="Targeting path location '$(_AppRefPath)' does not exist."
115+
<Message Importance="High"
116+
Text="Targeting path location '$(_AppRefPath)' does not exist and tests will use the SDK's bundled runtimes and targeting pack."
116117
Condition=" !$(_AppRefPathExists) AND '$(DoNotRequireSharedFxHelix)' == true " />
117-
<Error Text="Targeting path location '$(_AppRefPath)' does not exist."
118+
<Error Text="Targeting path location '$(_AppRefPath)' does not exist and some tests are guaranteed to fail."
118119
Condition=" !$(_AppRefPathExists) AND '$(DoNotRequireSharedFxHelix)' != true " />
119120

120121
<!-- Grab only the portions of the .dotnet/ tree built in this run and have Helix zip that up. -->

0 commit comments

Comments
 (0)