feat: Added .NET8 as TFM for Blazor.WebAssembly package

[bitsandfoxes]

Addresses #2021 (comment)

Since the package has a dependency on Microsoft.AspNetCore.Components.WebAssembly adding .NET 8 as TFM allows users to resolve the actual .NET 8 dependency chain.

[github-actions]

	Messages
📖	Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against ef63c49

[jamescrosswell]

LGTM

[klemmchr]

Should this be done for other libraries like Sentry.AspNetCore.Grpc as well?

[bitsandfoxes]

Should this be done for other libraries like Sentry.AspNetCore.Grpc as well?

As far as I can tell, either none of those have TFM version specific dependendencies or they are (like Sentry.AspNetCore) already targeting .NET8.

[klemmchr]

At least the mentioned package Sentry.AspNetCore.Grpc has a dependency on Sentry.AspNetCore which has TFM version specific dependencies. Same applies to Sentry.Google.Cloud.Functions. I didn't check all packages though.

[bitsandfoxes]

And we're building versions for those

sentry-dotnet/src/Sentry.AspNetCore/Sentry.AspNetCore.csproj

Line 4 in 5ff8acc

<TargetFrameworks>net8.0;net6.0</TargetFrameworks>

[klemmchr]

Sorry, I was unclear about what I mean. Since Sentry.AspNetCore has TFM version specific dependencies, dependant packages (like Sentry.AspNetCore.Grpc and Sentry.Google.Cloud.Functions) should build net8.0 as well but they don't at the moment. Unless I misunderstand the argument made for Blazor.WebAssembly.

[bitsandfoxes]

It's about the package's dependencies: Sentry.AspNetCore and Sentry.AspNetCore.Blazor.WebAssembly have TFM-dependent dependencies. I.e. Microsoft.AspNetCore.Components.WebAssembly v6 and v8. Same for Microsoft.Extensions.Configuration.Binder.

Looking through the other packages, they don't have those. So we don't need to target any additional TFMs explicitely.
Unless I'm getting that wrong somehow.

[klemmchr]

From my understanding this does not only apply to direct dependencies but also to downstream dependencies (see #2021 (comment)). This would affect all libraries that have a dependency on Sentry.AspNetCore too.

[bitsandfoxes]

TIL! Fair enough. Created an issue to track this: #3598