Update .NET to version 10 (#963)

Toshad · web-flow · commit 4b83df00caa5 · 2026-03-24T17:26:32.000+05:30
* Update .NET to version 10

Updating to latest .Net version since v8.0 is end-of-life
Also Fix .NET version in docker

* build fix

* build fix
diff --git a/.github/workflows/common.yml b/.github/workflows/common.yml
@@ -82,7 +82,7 @@ jobs:
           - name: Setup .NET
             uses: actions/setup-dotnet@v5
             with:
-                dotnet-version: 8.0.x
+                dotnet-version: 10.0.x
           - name: Restore dependencies
             run: dotnet restore
 
@@ -165,7 +165,7 @@ jobs:
           - name: Setup .NET
             uses: actions/setup-dotnet@v5
             with:
-                dotnet-version: 8.0.x
+                dotnet-version: 10.0.x
           - name: Restore dependencies
             run: dotnet restore
 
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -15,7 +15,7 @@ jobs:
         - name: Setup .NET
           uses: actions/setup-dotnet@v5
           with:
-              dotnet-version: 8.0.x
+              dotnet-version: 10.0.x
         - name: Restore dependencies
           run: dotnet restore
         - name: Build
diff --git a/ExtractorLauncher/.editorconfig b/ExtractorLauncher/.editorconfig
@@ -0,0 +1,5 @@
+[*.cs]
+
+# CA1515: Consider making public types internal
+# ExtractorLauncher is an application, not a library - public types are fine
+dotnet_diagnostic.CA1515.severity = none
diff --git a/ExtractorLauncher/ExtractorLauncher.csproj b/ExtractorLauncher/ExtractorLauncher.csproj
@@ -2,7 +2,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <OutputType>Exe</OutputType>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <RootNamespace>Cognite.OpcUa</RootNamespace>
     <AssemblyName>OpcuaExtractor</AssemblyName>
     <AnalysisMode>AllEnabledByDefault</AnalysisMode>
diff --git a/README.md b/README.md
@@ -14,7 +14,7 @@ with the dotnet CLI.
 
 ### Building
 
-Install .NET 8.0 from [here](https://dotnet.microsoft.com/download).
+Install .NET 10.0 from [here](https://dotnet.microsoft.com/download).
 
 To run:
 `dotnet run --project ExtractorLauncher/`.
@@ -87,7 +87,7 @@ The server will be reachable at `opc.tcp://localhost:62546`. Run `docker run --r
 
 ## Development
 
-You will need .NET 8.0. Then simply run `dotnet build` to compile,
+You will need .NET 10.0. Then simply run `dotnet build` to compile,
 or `dotnet run --project ExtractorLauncher` to compile and run.
 
 The compiler may complain about OpcUaExtractorSetup and OpcUaServiceManager, which aren't generally necessary to compile during development.
diff --git a/Server/.editorconfig b/Server/.editorconfig
@@ -0,0 +1,5 @@
+[*.cs]
+
+# CA1515: Consider making public types internal
+# Server is a test utility project, not a production library - public types are intentional
+dotnet_diagnostic.CA1515.severity = none
diff --git a/Server/Server.csproj b/Server/Server.csproj
@@ -2,7 +2,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <OutputType>Exe</OutputType>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <LangVersion>9.0</LangVersion>
     <AnalysisMode>AllEnabledByDefault</AnalysisMode>
   </PropertyGroup>
diff --git a/Test/.editorconfig b/Test/.editorconfig
@@ -0,0 +1,9 @@
+[*.cs]
+
+# CA1515: Consider making public types internal
+# Test project types don't need to be internal - they're not part of any public API
+dotnet_diagnostic.CA1515.severity = none
+
+# CA1873: Avoid expensive string formatting in logging
+# Not critical for test code - readability is more important
+dotnet_diagnostic.CA1873.severity = none
diff --git a/Test/CommonTestUtils.cs b/Test/CommonTestUtils.cs
@@ -38,7 +38,7 @@ You should have received a copy of the GNU General Public License
 [assembly: CLSCompliant(false)]
 namespace Test
 {
-    public static class CommonTestUtils
+    internal static class CommonTestUtils
     {
         private static readonly object portCounterLock = new object();
         private static int portCounter = 62200;
diff --git a/Test/Test.csproj b/Test/Test.csproj
@@ -1,7 +1,7 @@
 ﻿<?xml version="1.0" encoding="utf-8"?>
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFramework>net10.0</TargetFramework>
     <IsTestProject>true</IsTestProject>
     <IsPackable>false</IsPackable>
     <LangVersion>9.0</LangVersion>
diff --git a/Test/Unit/HistoryConsistencyTest.cs b/Test/Unit/HistoryConsistencyTest.cs
@@ -161,7 +161,7 @@ public async Task TestHistoryConsistencyData(
                 await CommonTestUtils.RunHistory(reader, states, HistoryReadType.BackfillData);
             }
 
-            var distinct = (await queue.DrainAsync().ToListAsync(default)).DistinctBy(dp => (dp.Id, dp.Timestamp)).GroupBy(dp => dp.Id)
+            var distinct = (await ApiUtils.ToListAsync(queue.DrainAsync(), default)).DistinctBy(dp => (dp.Id, dp.Timestamp)).GroupBy(dp => dp.Id)
                 .ToDictionary(group => group.Key, group => group.ToArray());
 
             Assert.Equal(6, distinct.Count);
@@ -244,7 +244,7 @@ public async Task TestHistoryConsistencyEvents(
                 await CommonTestUtils.RunHistory(reader, states, HistoryReadType.BackfillEvents);
             }
 
-            var distinct = (await queue.DrainAsync().ToListAsync(default)).DistinctBy(evt => evt.Message).GroupBy(evt => evt.EmittingNode)
+            var distinct = (await ApiUtils.ToListAsync(queue.DrainAsync(), default)).DistinctBy(evt => evt.Message).GroupBy(evt => evt.EmittingNode)
                 .ToDictionary(group => group.Key, group => group.ToArray());
 
             Assert.Equal(2, distinct.Count);
diff --git a/Test/Utils/StaticServerTestFixture.cs b/Test/Utils/StaticServerTestFixture.cs
@@ -1,8 +1,10 @@
-﻿using Xunit;
+﻿using System.Diagnostics.CodeAnalysis;
+using Xunit;
 
 namespace Test.Utils
 {
     [CollectionDefinition("Shared server tests")]
+    [SuppressMessage("Design", "CA1515:Consider making public types internal", Justification = "xUnit requires collection definitions to be public")]
     public class SharedServerTests : ICollectionFixture<StaticServerTestFixture>
     {
     }
diff --git a/build-zip-server.sh b/build-zip-server.sh
@@ -6,7 +6,7 @@ linux="$2"
 version="$3"
 
 mkdir -p $configuration/
-mv Server/bin/Release/net8.0/$configuration/publish/* ./$configuration/
+mv Server/bin/Release/net10.0/$configuration/publish/* ./$configuration/
 rm -f ./$configuration/*.config ./$configuration/*.pdb ./$configuration/*.xml
 
 dotnet sbom-tool generate -b ./$configuration/ -bc Server/ -pn Server -pv $version -ps Cognite -nsb https://sbom.cognite.com
diff --git a/build-zip.sh b/build-zip.sh
@@ -6,7 +6,7 @@ linux="$2"
 version="$3"
 
 mkdir -p $configuration/
-mv ExtractorLauncher/bin/Release/net8.0/$configuration/publish/* ./$configuration/
+mv ExtractorLauncher/bin/Release/net10.0/$configuration/publish/* ./$configuration/
 rm -f ./$configuration/*.config ./$configuration/*.pdb ./$configuration/*.xml
 mkdir -p ./$configuration/config
 

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ You should have received a copy of the GNU General Public License`
`38`	`38`	`[assembly: CLSCompliant(false)]`
`39`	`39`	`namespace Test`
`40`	`40`	`{`
`41`		`- public static class CommonTestUtils`
	`41`	`+ internal static class CommonTestUtils`
`42`	`42`	`{`
`43`	`43`	`private static readonly object portCounterLock = new object();`
`44`	`44`	`private static int portCounter = 62200;`