Merge pull request #1 from islero/dependabot/github_actions/github/co… #25
isleroAnnotations
10 warnings
|
Analyze:
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.
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
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)
|
|
Analyze:
src/Backtest.Net/Executors/BacktestingNetExecutor.cs#L24
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Analyze:
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.
|