Skip to content

Merge XPEvo fixed/changes into main repository #545

Merge XPEvo fixed/changes into main repository

Merge XPEvo fixed/changes into main repository #545

Triggered via pull request November 9, 2024 16:13
Status Success
Total duration 1m 38s
Artifacts 1

dotnet.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

18 warnings
build_and_test
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3, actions/upload-artifact@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build_and_test: src/EvoSC.Modules.SourceGeneration/AssemblyModuleInfoGenerator.cs#L15
'EvoSC.Modules.SourceGeneration.AssemblyModuleInfoGenerator': A project containing analyzers or source generators should specify the property '<EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>'
build_and_test: src/EvoSC.Modules.SourceGeneration/AssemblyModuleInfoGenerator.cs#L89
Loop should be simplified by calling Select(dependency => dependency.Key)) (https://rules.sonarsource.com/csharp/RSPEC-3267)
build_and_test: src/EvoSC.Common/Config/Stores/TomlConfigStore.cs#L22
Converting null literal or possible null value to non-nullable type.
build_and_test: src/EvoSC.Common/Config/Stores/TomlConfigStore.cs#L26
Possible null reference argument for parameter 'path' in 'DirectoryInfo Directory.CreateDirectory(string path)'.
build_and_test: src/EvoSC.Common/Controllers/Context/GenericControllerContext.cs#L25
Non-nullable property 'ServiceScope' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build_and_test: src/EvoSC.Common/Controllers/Context/GenericControllerContext.cs#L25
Non-nullable property 'Controller' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build_and_test: src/EvoSC.Common/Controllers/Context/GenericControllerContext.cs#L25
Non-nullable property 'AuditEvent' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build_and_test: src/EvoSC.Common/Controllers/Context/GenericControllerContext.cs#L30
Non-nullable property 'Controller' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build_and_test: src/EvoSC.Common/Controllers/Context/GenericControllerContext.cs#L30
Non-nullable property 'AuditEvent' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
build_and_test: src/EvoSC.Common/Controllers/EvoScController.cs#L10
Non-nullable property 'Context' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "PR_NUMBER". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
Redundant using directive: tests/Modules/RoundRankingModule.Tests/Controllers/RoundRankingEventControllerTests.cs#L4
Using directive is not required by the code and can be safely removed
Redundant using directive: tests/Modules/RoundRankingModule.Tests/Controllers/RoundRankingEventControllerTests.cs#L9
Using directive is not required by the code and can be safely removed
Unused local variable: src/Modules/RoundRankingModule/Services/RoundRankingService.cs#L39
Local variable 'removedCheckpointData' is never used
Unused local variable: src/Modules/RoundRankingModule/Services/RoundRankingService.cs#L58
Local variable 'removedCheckpointData' is never used

Artifacts

Produced during runtime
Name Size
PR_NUMBER
4 Bytes