Build: Drop support for .NET 7

[danielchalmers]

Description

.NET 7 reached End of Support on May 14, 2024.

Todo:

MudBlazor/src/MudBlazor/Utilities/Debounce/DebounceDispatcher.cs

Lines 42 to 45 in f317083

	// ReSharper disable MethodHasAsyncOverload (not available in .net7)
	// Cancel the previous debounce task if it exists
	_cancellationTokenSource?.Cancel();
	// ReSharper restore MethodHasAsyncOverload

MudBlazor/src/MudBlazor/Services/Popover/PopoverService.cs

Lines 227 to 243 in f317083

	// ReSharper disable once MethodHasAsyncOverload - not available in .NET6 & .NET7
	_cancellationTokenSource.Cancel();
	await DestroyPopoversQuick();
	_batchExecutor.Dispose();
	// In case someone has custom implementation and didn't unsubscribe
	_observerManager.Clear();
	_popoverSemaphore.Dispose();
	// https://github.com/MudBlazor/MudBlazor/pull/5367#issuecomment-1258649968
	// Fixed in NET8
	// Do not send CancellationToken as it was cancelled.
	#pragma warning disable CA2012 // Use ValueTasks correctly
	_ = _popoverJsInterop.Dispose();
	#pragma warning restore CA2012 // Use ValueTasks correctly

MudBlazor/src/MudBlazor/Components/ThemeProvider/MudThemeProvider.razor.cs

Lines 478 to 479 in f317083

	// When .NET7 is dropped we can use async Dispose, but for now MAUI has bug https://github.com/MudBlazor/MudBlazor/pull/5367#issuecomment-1258649968.
	_ = JsRuntime.InvokeVoidAsyncIgnoreErrors("stopWatchingDarkThemeMedia");

MudBlazor/src/MudBlazor/Services/Browser/BrowserViewportService.cs

Lines 280 to 282 in f317083

	// https://github.com/MudBlazor/MudBlazor/pull/5367#issuecomment-1258649968
	// Fixed in NET8
	_ = _resizeListenerInterop.Dispose();

MudBlazor/src/MudBlazor/Services/Popover/PopoverService.cs

Lines 238 to 243 in f317083

	// https://github.com/MudBlazor/MudBlazor/pull/5367#issuecomment-1258649968
	// Fixed in NET8
	// Do not send CancellationToken as it was cancelled.
	#pragma warning disable CA2012 // Use ValueTasks correctly
	_ = _popoverJsInterop.Dispose();
	#pragma warning restore CA2012 // Use ValueTasks correctly

MudBlazor/src/MudBlazor/Utilities/AsyncKeyedLock/AsyncKeyedLocker.cs

Line 737 in f317083

#region AsynchronousTryNet8.0

How Has This Been Tested?

Type of Changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation (fix or improvement to the website or code docs)

Checklist

• The PR is submitted to the correct branch (dev).
• My code follows the code style of this project.
• I've added relevant tests.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.56%. Comparing base (28bc599) to head (b72ef03).
Report is 525 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #9511      +/-   ##
==========================================
+ Coverage   89.82%   90.56%   +0.73%     
==========================================
  Files         412      406       -6     
  Lines       11878    12708     +830     
  Branches     2364     2462      +98     
==========================================
+ Hits        10670    11509     +839     
+ Misses        681      638      -43     
- Partials      527      561      +34     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[xC0dex]

Side note: I think the FrozenSet is not the best type here to be used compared to Hashset because it requires more time and allocations to be created.

Method	Mean	Error	StdDev	Ratio	RatioSD	Allocated	Alloc Ratio
ToHashSet	2.242 us	0.0150 us	0.0125 us	1.00	0.00	2.23 KB	1.00
ToFrozenSet	48.176 us	0.1614 us	0.1431 us	21.48	0.14	12.45 KB	5.57

[danielchalmers]

@ScarletKuro

[ScarletKuro]

I guess it makes sense to have HashSet in this place as we just enumerate it and we do not do read individual values where the FrozenSet is faster.

[ScarletKuro]

See: dotnet/aspnetcore#54543 (comment)

[ScarletKuro]

Sticky note:

1. MudBlazor/src/MudBlazor/Components/Table/EditButtonContext.cs

   Line 7 in c031655

   namespace MudBlazorFix // A bug in Blazor requires a different namespace in some scenarios, see: https://github.com/dotnet/aspnetcore/issues/36326 (fixed in .NET 7)

2. Fix: Server Side DataGrid EventListener Memory Leak #9735

[henon]

Thanks @danielchalmers ! Work on v8 is now officially commencing!

[ScarletKuro]

Added to v8.0.0 Migration Guide #9953