Update to net9.0 #11000
Update to net9.0 #11000
Conversation
| @@ -63,7 +63,6 @@ | |||
| <PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.FileSystemGlobbing" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.NonCapturingTimer.Sources" Version="5.0.0-preview.4.20205.1" /> | |||
There was a problem hiding this comment.
Yeah, this source package would need more warnings to be suppressed with net9 (I'm not yet sure why, but some existing IDE* error codes started newly being reported with net9.0). It seems better to just include the one source file from this source package directly and make it conform to our editorconfig. I also fixed up some missing file headers that got missed because of the IDE0073 being ignored previously.
There was a problem hiding this comment.
It looks like this is only used in one place in the MS.VS.Razor.LegacyEditor proejct. Could you move this file there?
| @@ -22,8 +24,15 @@ public CodeGenerationIntegrationTest(bool designTime = false) | |||
| : base(layer: TestProject.Layer.Compiler) | |||
| { | |||
| this.designTime = designTime; | |||
| BaseCompilation = BaseCompilation.AddReferences( | |||
| MetadataReference.CreateFromFile(typeof(TestTagHelperDescriptors).Assembly.Location)); | |||
There was a problem hiding this comment.
This assembly reference started failing due to now being net9.0 but others (via Basic.Reference.Assemblies) being net8.0.
Refactoring of the following file (TestTagHelperDescriptors.cs) is related to this.
| @@ -63,7 +63,6 @@ | |||
| <PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.FileSystemGlobbing" Version="$(_MicrosoftExtensionsPackageVersion)" /> | |||
| <PackageVersion Include="Microsoft.Extensions.NonCapturingTimer.Sources" Version="5.0.0-preview.4.20205.1" /> | |||
There was a problem hiding this comment.
Yeah, this source package would need more warnings to be suppressed with net9 (I'm not yet sure why, but some existing IDE* error codes started newly being reported with net9.0). It seems better to just include the one source file from this source package directly and make it conform to our editorconfig. I also fixed up some missing file headers that got missed because of the IDE0073 being ignored previously.
| @@ -77,7 +77,7 @@ | |||
| --> | |||
| <Otherwise> | |||
| <PropertyGroup> | |||
| <DefaultNetCoreTargetFramework>net8.0</DefaultNetCoreTargetFramework> | |||
| <DefaultNetCoreTargetFramework>net9.0</DefaultNetCoreTargetFramework> | |||
There was a problem hiding this comment.
Hm, I guess it's not that simple, parts that deploy to VS/VSCode need to stay on net8.0. I will at least extract the independent parts of this PR for now.
There was a problem hiding this comment.
To make it more of a gradual change, you could add net9 to the multi-targeting, and we can continue to change it as the needs of SDK/VS/VSCode change. There are some tradeoffs to making the multi-targeting conditional to being on a CI machine or not.
Doing this in roslyn (dotnet/roslyn#75465) and didn't want to leave razor behind.