er fps patch #476
Vinjul1704Annotations
22 warnings
|
Non-accessed local variable:
src/Cli/Program.cs#L50
Local variable 'init' is only assigned but its value is never used
|
|
Redundant member initializer:
src/SoulSplitter/Splitters/EldenRingSplitter.cs#L177
Initializing field by default value is redundant
|
|
Redundant member initializer:
src/SoulSplitter/Splitters/EldenRingSplitter.cs#L176
Initializing field by default value is redundant
|
|
Redundant member initializer:
src/SoulSplitter/UI/EldenRing/EldenRingViewModel.cs#L48
Initializing field by default value is redundant
|
|
Redundant using directive:
src/SoulMemory/soulmods/Soulmods.cs#L21
Using directive is not required by the code and can be safely removed
|
|
Redundant using directive:
src/Cli/Program.cs#L33
Using directive is not required by the code and can be safely removed
|
|
Redundant using directive:
src/SoulSplitter/SoulComponent.cs#L35
Using directive is not required by the code and can be safely removed
|
|
Redundant using directive:
src/Cli/Program.cs#L25
Using directive is not required by the code and can be safely removed
|
|
Redundant using directive:
src/SoulSplitter/Splitters/EldenRingSplitter.cs#L29
Using directive is not required by the code and can be safely removed
|
|
Unused local variable:
src/SoulMemory/Native/Kernel32.cs#L370
Local variable 'eatPtr' is never used
|
|
build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v2, actions/cache@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@v3, actions/setup-java@v2, actions/cache@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
build:
src/SoulMemory/Native/Kernel32.cs#L82
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
build:
src/SoulMemory/Native/Kernel32.cs#L81
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
build:
src/SoulMemory/soulmods/Soulmods.cs#L120
Remove the unused local variable 'data'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
|
|
build:
src/SoulMemory/Native/Kernel32.cs#L283
Remove the unused local variable 'len'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
|
|
build:
src/SoulMemory/Native/Kernel32.cs#L370
Remove the unused local variable 'eatPtr'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
|
|
build:
src/SoulMemory/Native/Kernel32.cs#L377
Remove the unused local variable 'ordinal'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
|
|
build:
src/SoulMemory/EldenRing/EldenRing.cs#L126
Remove this empty 'case' clause. (https://rules.sonarsource.com/csharp/RSPEC-3458)
|
|
build:
src/SoulMemory/EldenRing/EldenRing.cs#L395
Refactor this method to reduce its Cognitive Complexity from 20 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
build:
src/soulsplitter/Splitters/EldenRingSplitter.cs#L186
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
|
build:
src/soulsplitter/Splitters/EldenRingSplitter.cs#L230
Remove this commented out code. (https://rules.sonarsource.com/csharp/RSPEC-125)
|
Artifacts
Produced during runtime
| Name | Size | |
|---|---|---|
|
SoulSplitter
|
13 MB |
|