update warning presence in Microsoft.Build.Locator.targets

[YuliiaKovalova]

Fixes: #89

[Rainer Sigwald (rainersigwald)]

Does this work? It looks like it should hit dotnet/msbuild#1155

[Ladi Prosek (ladipro)]

Does this work? It looks like it should hit dotnet/msbuild#1155

+1, this does not build.

YuliiaKovalova, in addition to fixing it could you please also add

  <Import Project="..\..\src\MSBuildLocator\build\Microsoft.Build.Locator.targets"/>

to BuilderApp.csproj so this file is covered by CI?

[YuliiaKovalova]

Does this work? It looks like it should hit dotnet/msbuild#1155

+1, this does not build.

YuliiaKovalova, in addition to fixing it could you please also add

  <Import Project="..\..\src\MSBuildLocator\build\Microsoft.Build.Locator.targets"/>

to BuilderApp.csproj so this file is covered by CI?

I was inspired by https://stackoverflow.com/questions/3289538/is-there-any-msbuild-task-to-check-if-a-string-contains-another-string-similar

:) I will doublecheck

Should I check it with a regular expression?

[Ladi Prosek (ladipro)]

I was inspired by https://stackoverflow.com/questions/3289538/is-there-any-msbuild-task-to-check-if-a-string-contains-another-string-similar

Yes, it works for properties but sadly not for metadata.

Should I check it with a regular expression?

I don't think so. Please check this comment.

[YuliiaKovalova]

I was inspired by https://stackoverflow.com/questions/3289538/is-there-any-msbuild-task-to-check-if-a-string-contains-another-string-similar

Yes, it works for properties but sadly not for metadata.

Should I check it with a regular expression?

I don't think so. Please check this comment.

It works, thank you !

[Ladi Prosek (ladipro)]

LGTM. Please wait for Rainer Sigwald (@rainersigwald)'s approval before merging.

[YuliiaKovalova]

Rainer Sigwald (@rainersigwald) , please review the changes.

[Rainer Sigwald (rainersigwald)]

Sorry, I seem to have totally missed this from a couple of weeks ago. LGTM!