Skip to content

ci: bump actions/cache from 4 to 5 #21

ci: bump actions/cache from 4 to 5

ci: bump actions/cache from 4 to 5 #21

Triggered via pull request December 27, 2025 11:08
Status Success
Total duration 1m 5s
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/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: 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/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/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/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/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/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/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:dc6d7899bab4c33f62f560d759f457fd710bbcc21c7423158a638bf97f123012
test-results Expired
35.2 KB
sha256:5a73a283a16fde07f36f5af36dd8bb0e658cd43a34e62e0f095a98191e68d5d2