[Tests] Remove Builder.UseDotNet#8441
Conversation
91f3461 to
d735498
Compare
|
This should be ready for review. Since there is a lot changing/being removed here we may want to hold off on merging for a little while however, if only to make any upcoming backports needed for .NET 8 easier. |
...amarin.Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinAndroidCommonProject.cs
Outdated
Show resolved
Hide resolved
a9e1720 to
7c9f315
Compare
|
There's one issue that's been happening here consistently that I can't seem to track down, and couldn't repro locally: I don't think a version of |
|
|
|
@pjcollins we'd load it if the |
Ooh great thanks, I think this explains it. It looks like this test is running during the same job as |
3bf7df2 to
1eb0f83
Compare
src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/AndroidUpdateResourcesTest.cs
Show resolved
Hide resolved
|
I think this should be good to go whenever there's some time to review. |
...Android.Build.Tasks/Tests/Xamarin.ProjectTools/Android/XamarinFormsMapsApplicationProject.cs
Outdated
Show resolved
Hide resolved
a6b5521 to
25ec969
Compare
|
Not yet sure what's up with the latest resource designer test failures but they seem to be happening on other PRs, and I haven't been able to repro them locally. |
The .NET versus Classic switches have been removed from the MSBuild tests. Tests / asserts that only applied to classic have been removed. Test parameters that only applied to classic (`aapt` vs `aapt2`) have been removed. Some project properties that only applied to classic (`UseLatestPlatformSdk`) have been removed.
25ec969 to
4c0d311
Compare
|
@pjcollins the designer tests have been fixed, if you rebase they should go away 👍 |
* main: [tests] Remove Builder.UseDotNet (#8441)

The .NET versus Classic switches have been removed from the MSBuild
tests. Tests / asserts that only applied to classic have been removed.
Test parameters that only applied to classic (
aaptvsaapt2) havebeen removed. Some project properties that only applied to classic
(
UseLatestPlatformSdk) have been removed.