ci: bump github/codeql-action from 3 to 4 #1
Annotations
10 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/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:
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/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:
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/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/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/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.
|
Loading