Skip to content

ci: bump github/codeql-action from 3 to 4 #18

ci: bump github/codeql-action from 3 to 4

ci: bump github/codeql-action from 3 to 4 #18

Triggered via pull request December 27, 2025 11:08
Status Success
Total duration 1m 12s
Artifacts 2

ci.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

19 warnings
Build: src/Backtest.Net/Engines/EngineV9.cs#L10
Parameter 'bool useFullCandleForCurrent' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
Build: tests/Backtest.Net.Tests/SymbolDataSplitterTests/SymbolDataSplitterHeavyTests.cs#L44
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
Build: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L332
Use the "Count" property instead of Enumerable.Count() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1829)
Build: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L105
Use the "Count" property instead of Enumerable.Count() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1829)
Build: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L101
Prefer comparing 'Count' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
Build: src/Backtest.Net/Engines/EngineV8.cs#L50
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Build: src/Backtest.Net/Engines/EngineV10.cs#L46
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Build: src/Backtest.Net/Engines/EngineV9.cs#L38
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Build: src/Backtest.Net/Executors/BacktestingNetExecutor.cs#L24
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Build: src/Backtest.Net/Engines/EngineV9.cs#L10
Parameter 'bool useFullCandleForCurrent' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.
Test: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L332
Use the "Count" property instead of Enumerable.Count() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1829)
Test: tests/Backtest.Net.Tests/SymbolDataSplitterTests/SymbolDataSplitterHeavyTests.cs#L44
Test methods should not use blocking task operations, as they can cause deadlocks. Use an async test method and await instead. (https://xunit.net/xunit.analyzers/rules/xUnit1031)
Test: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L105
Use the "Count" property instead of Enumerable.Count() (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1829)
Test: tests/Backtest.Net.Tests/EngineTests/EngineTestsV2.cs#L101
Prefer comparing 'Count' to 0 rather than using 'Any()', both for clarity and for performance (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860)
Test: src/Backtest.Net/Engines/EngineV8.cs#L50
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Test: src/Backtest.Net/Engines/EngineV10.cs#L46
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Test: src/Backtest.Net/Engines/EngineV9.cs#L38
Use 'ThrowIfCancellationRequested' instead of checking 'IsCancellationRequested' and throwing 'OperationCanceledException' (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2250)
Test: src/Backtest.Net/Executors/BacktestingNetExecutor.cs#L24
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
Test: src/Backtest.Net/Engines/EngineV9.cs#L10
Parameter 'bool useFullCandleForCurrent' is captured into the state of the enclosing type and its value is also passed to the base constructor. The value might be captured by the base class as well.

Artifacts

Produced during runtime
Name Size Digest
build-artifacts Expired
13.1 MB
sha256:3247f1b9db9be7a88b126e1f1af12890a014f638eed52ebb2c66d842b75596b6
test-results Expired
35 KB
sha256:9c6fe932ed6a76d8a42c4481c8eb0f87829cdbf01e255341a0dba9cb3a6fcfc8