Skip to content

[Add] java 17 to SonarQube GH action #1595

[Add] java 17 to SonarQube GH action

[Add] java 17 to SonarQube GH action #1595

Triggered via push December 11, 2023 11:12
Status Failure
Total duration 2m 42s
Artifacts

CodeQuality.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

1 error and 11 warnings
Build
Process completed with exit code 1.
Build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-java@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Build: COMET.Web.Common/App.razor.cs#L92
Dispose 'cancellationTokenSource' when it is no longer needed. (https://rules.sonarsource.com/csharp/RSPEC-2930)
Build: COMET.Web.Common/Utilities/DisposableObject/DisposableObject.cs#L33
Fix this implementation of 'IDisposable' to conform to the dispose pattern. (https://rules.sonarsource.com/csharp/RSPEC-3881)
Build: COMET.Web.Common/Components/DisposableComponent.razor.cs#L33
Fix this implementation of 'IDisposable' to conform to the dispose pattern. (https://rules.sonarsource.com/csharp/RSPEC-3881)
Build: COMET.Web.Common/Extensions/GuidExtensions.cs#L143
Split this method into two, one handling parameters check and the other handling the iterator. (https://rules.sonarsource.com/csharp/RSPEC-4456)
Build: COMET.Web.Common/Extensions/SessionServiceExtensions.cs#L69
The parameter name 'Session' is not declared in the argument list. (https://rules.sonarsource.com/csharp/RSPEC-3928)
Build: COMET.Web.Common/Extensions/SessionServiceExtensions.cs#L50
Split this method into two, one handling parameters check and the other handling the asynchronous code. (https://rules.sonarsource.com/csharp/RSPEC-4457)
Build: COMET.Web.Common/ViewModels/Components/ParameterEditors/EnumerationParameterTypeEditorViewModel.cs#L121
Change this condition so that it does not always evaluate to 'True'. (https://rules.sonarsource.com/csharp/RSPEC-2589)
Build: COMET.Web.Common/Components/ParameterTypeEditors/CompoundParameterTypeEditor.razor#L35
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
Build: COMETwebapp/Program.cs#L63
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
Build: COMETwebapp/Components/Shared/About.razor.cs#L39
Refactor your code not to use hardcoded absolute paths or URIs. (https://rules.sonarsource.com/csharp/RSPEC-1075)