Skip to content
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

Warning fixes: spring wave #15747

Merged
merged 11 commits into from
May 21, 2024
Merged

Warning fixes: spring wave #15747

merged 11 commits into from
May 21, 2024

Conversation

MrJul
Copy link
Member

@MrJul MrJul commented May 16, 2024

It's spring, warnings again are blooming, now's the time for fixing!

Noteworthy changes:

  • Base: remove internal Obsolete attributes on ValueStore (with @grokys' approval).
  • RenderTests: updated ImageSharp to 2.1.8 (only used in tests, but has known vulnerabilities).
  • Browser: only enable PThread if WasmEnableThreads is true.
  • ControlCatalog.Desktop: reference Win32+Skia directly to avoid bringing incompatible packages to net461.

@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0048509-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

Comment on lines +17 to +19
<Compile Include="..\..\..\src\Avalonia.Base\Metadata\NullableAttributes.cs" Link="Compatibility\NullableAttributes.cs" />
<Compile Include="..\..\..\src\Avalonia.Base\Compatibility\TrimmingAttributes.cs" Link="Compatibility\TrimmingAttributes.cs" />
<Compile Include="..\..\..\src\Shared\IsExternalInit.cs" Link="Compatibility\IsExternalInit.cs" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to unify how these pollyfils are referenced. Somewhere it's part of Avalonia.Base + internals visible to, and somewhere it's a shared file.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what's better. But shared file is more annoying when you already have internals visible to enabled.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't really unify, as much as I'd like that.

As you mentioned, the InternalsVisibleTo mandates that we use the types from Avalonia.Base if we don't want spurious warnings for conflicting attributes (like the ones this PR fixes). But a few projects, such as Avalonia.Designer.HostApp aren't dependent of Avalonia.Base, so we have to use the shared files instead.

Should be resolved if/when we drop netstandard2.0, hopefully before 2030 ;)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:)
Screenshot 2024-05-20 at 9 10 47 PM

@maxkatz6 maxkatz6 enabled auto-merge May 20, 2024 12:10
@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0048539-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 added this pull request to the merge queue May 21, 2024
Merged via the queue into AvaloniaUI:master with commit b30894c May 21, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants