-
Notifications
You must be signed in to change notification settings - Fork 128
Remove special logic for TypeConverterAttribute #2659
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
Conversation
This fixes the issue, but I couldn't reproduce the issue in the test, so there's no checks for regressions.
test/Mono.Linker.Tests.Cases/ComponentModel/TypeDescriptionProviderAttributeOnType.cs
Outdated
Show resolved
Hide resolved
Are the annotations correct in .NET6 release so targeting LTS from net7 works as expected? |
Yes. See dotnet/runtime#39144. TypeConverterAttribute was made trim-compatible in .NET 5. @agocke @vitek-karas @jtschuster - Is there any chance this change can be backported to .NET 6 for Maui? I'm seeing the following warnings 330 times when trimming a Maui app, which is causing A LOT of noise.
In case you are curious, I explained the reason these warnings occur here. |
Given that this is an SDK-only change with high impact, I think it has a chance. |
(cherry picked from commit b10e1bc)
Fixes #1942
We no longer need special logic for
TypeConverterAttribute
s. This removes that code, and adds a test to reproduce the issue found in #1942.