ci: bump actions/cache from 4 to 5 #21
Annotations
9 warnings
|
Test with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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 with coverage:
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