Skip to content

Commit 50317ec

Browse files
author
MPCoreDeveloper
committed
fix(sonar): correct NOSONAR placement for S2068/S2368
1 parent e8e67dc commit 50317ec

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/SharpCoreDB/DataStructures/Table.CRUD.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ public long[] InsertBatch(List<Dictionary<string, object>> rows)
276276
/// Semantics are identical (validation, defaults, NOT NULL, PK, indexes, engine batch).
277277
/// </summary>
278278
[MethodImpl(MethodImplOptions.AggressiveOptimization)]
279-
internal long[] InsertBatch(object[][] rows) => InsertBatch(rows, this.Columns);
279+
internal long[] InsertBatch(object[][] rows) => InsertBatch(rows, this.Columns); // NOSONAR:S2368 - internal API consumed by the SQL batch-INSERT parser
280280

281281
/// <summary>
282282
/// Dedicated SQL batch-INSERT fast path with an explicit user-facing column order (as used

tools/SharpCoreDB.AotSmoke/Program.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@
1919
using Microsoft.Extensions.DependencyInjection;
2020
using SharpCoreDB;
2121

22-
// NOSONAR:S2068 - intentional: smoke-test tool uses a throwaway local password (never a real credential).
23-
const string AotPassword = "aot123";
22+
// Smoke-test tool: throwaway local password, never a real credential.
23+
const string AotPassword = "aot123"; // NOSONAR:S2068 - intentional test fixture value
2424

2525
var services = new ServiceCollection();
2626
services.AddSharpCoreDB();

0 commit comments

Comments
 (0)