dark icon indicating copy to clipboard operation
dark copied to clipboard

Upgrade to .NET 7

Open StachuDotNet opened this issue 3 years ago • 5 comments

Do we wait for full release (~November) or jump earlier? FWIW, per https://github.com/dotnet/runtime/issues/70286#issuecomment-1156652738 the .Result that we're currently doing in Analysis.fs/Wasm.fs should be removable with .NET 7 upgrade.

(I'm not pushing to go upgrade early - just raising the question)

  • [ ] Review all DOTNET7TODO

StachuDotNet avatar Jun 15 '22 16:06 StachuDotNet

Personally I don't like to upgrade before a "release candidate" version, if we can help it. I'm not strongly opposed to doing an upgrade if it's required before then, though involves doubling the upgrade work.

pbiggar avatar Jun 15 '22 16:06 pbiggar

We removed the .Result issue, however the following issue remains:

Screen Shot 2022-09-05 at 5 01 43 PM

Do you think that this issue will also be solved by moving to .Net 7?

pbiggar avatar Sep 05 '22 21:09 pbiggar

I'm unsure, but suspect it would be - the .net team has noted general improvements to this sort of functionality here and would imagine this would be part of that. Maybe I'm conflating 2 issues though

StachuDotNet avatar Sep 05 '22 21:09 StachuDotNet

OK, I'm just going to hide the error for now then

pbiggar avatar Sep 05 '22 21:09 pbiggar

Right I certainly think it's worth waiting until the .NET 7 upgrade and circling back here only if that doesn't resolve the issue.

StachuDotNet avatar Sep 05 '22 21:09 StachuDotNet

Done today!

pbiggar avatar Mar 01 '23 04:03 pbiggar