Skip to content

Commit 7487cdb

Browse files
Fix parameterized query bug and update CI for .NET 10.0
- Fixed SqlParser.Execute to pass bound SQL instead of original SQL with placeholders - Updated CI workflow from .NET 9.0.x to .NET 10.0.x - Added Demo CLI run step to CI workflow Co-authored-by: MPCoreDeveloper <37024522+MPCoreDeveloper@users.noreply.github.com>
1 parent 026fe9e commit 7487cdb

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,12 @@ jobs:
1515
- name: Setup .NET
1616
uses: actions/setup-dotnet@v4
1717
with:
18-
dotnet-version: 9.0.x
18+
dotnet-version: 10.0.x
1919
- name: Restore dependencies
2020
run: dotnet restore
2121
- name: Build
2222
run: dotnet build --no-restore
2323
- name: Test
24-
run: dotnet test --no-build --verbosity normal
24+
run: dotnet test --no-build --verbosity normal
25+
- name: Run Demo CLI
26+
run: dotnet run --project SharpCoreDB.Demo --no-build

SharpCoreDB/Services/SqlParser.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public void Execute(string sql, Dictionary<string, object?> parameters, IWAL? wa
6262

6363
// Proceed with existing logic
6464
var parts = sql.Trim().Split(' ', StringSplitOptions.RemoveEmptyEntries);
65-
this.ExecuteInternal(originalSql ?? sql, parts, wal, originalSql ?? sql);
65+
this.ExecuteInternal(sql, parts, wal, originalSql ?? sql);
6666
}
6767

6868
/// <summary>

0 commit comments

Comments
 (0)