Revert "Move tests to target .NET 7"

[mavasani]

Reverts #6364 to fix the build break in the repo

[Youssef1313]

@mavasani Instead of reverting the PR, could you try targeting net8.0 instead?
I think the issue is that .NET 7 runtime isn't found starting with #6409

[mavasani]

Okay, let me try on #6428

[Youssef1313]

I'm also curious about the move to .NET 8 SDK too early?

There is no installer available in https://dotnet.microsoft.com/en-us/download, so we have to install nightly builds from https://github.com/dotnet/installer#table? This should at least by reflected in CONTRIBUTING.md if usage of .NET 8 SDK is kept

[mavasani]

I'm also curious about the move to .NET 8 SDK too early?

Do you know which PR did that? I agree we should stay on .NET 7

[mavasani]

I'm also curious about the move to .NET 8 SDK too early?

Do you know which PR did that? I agree we should stay on .NET 7

Seems like it happened with #6409

[Youssef1313]

@mavasani Do we want to revert #6409 too?

[codecov]

Codecov Report

Merging #6429 (356147f) into main (7b6ad7e) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6429   +/-   ##
=======================================
  Coverage   96.14%   96.15%           
=======================================
  Files        1365     1365           
  Lines      317494   317494           
  Branches    10269    10269           
=======================================
+ Hits       305269   305273    +4     
+ Misses       9791     9787    -4     
  Partials     2434     2434           

[mavasani]

Force merging to fix the build break

[mavasani]

@mavasani Do we want to revert #6409 too?

@jmarolf?