-
-
Notifications
You must be signed in to change notification settings - Fork 887
Add .NET10.0 as TFM #3014
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add .NET10.0 as TFM #3014
Conversation
Can not reproduce it locally..
|
|
||
| Assert.Equal(0, count); | ||
| Assert.Equal([], output); | ||
| Assert.Equal(Array.Empty<byte>(), output); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn’t need to do this. Might be something to do with breaking changes re span overloads though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that came from new overloads
|
I got some problems on my machine.. One with libgdiplus and all the tests that use System.Drawing as a reference. The other is a Tiff test. --> #3017 When I use a tolerant comparer the test is green. Now I am only struggling with the FeatureTestRunner. Now going to look into it |
Tried to debug it on my side. On my machine DOTNET_EnableArm64AdvSimd=0 seems to not work. And on CI/CD we have a problem with DisableSSE .... strange... |
|
I made a small repro and created an issue for this in the runtime repo |
@JimBobSquarePants I got an answer there. The grouping changed and you can now only disable groups and not the individual ones.. What is your favorite way to go on with this? Introduce many #if NET10_0_OR_GREATER in the test code. Or disable the groups for net8 and net10? |
e4f9727 to
6aad0e7
Compare
tests/ImageSharp.Tests/Formats/WebP/ColorSpaceTransformUtilsTests.cs
Outdated
Show resolved
Hide resolved
tests/ImageSharp.Tests/Processing/Processors/Convolution/BokehBlurTest.cs
Outdated
Show resolved
Hide resolved
| <Choose> | ||
| <When Condition="$(SIXLABORS_TESTING_PREVIEW) == true"> | ||
| <PropertyGroup> | ||
| <TargetFrameworks>net8.0;net9.0</TargetFrameworks> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JimBobSquarePants Should we add .net10.0 to the "Otherwise" section too? So the next release would be build against both LTS versions of .NET?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, just 8. I want to always target a single framework
No description provided.