Skip to content

Add ParameterGroupName to ParameterEditor; fixes #720 #2548

Add ParameterGroupName to ParameterEditor; fixes #720

Add ParameterGroupName to ParameterEditor; fixes #720 #2548

Triggered via pull request December 6, 2024 10:17
Status Success
Total duration 3m 13s
Artifacts

CodeQuality.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L76
Make sure the use of 'this' doesn't expose partially-constructed instances of this class in multi-threaded environments. (https://rules.sonarsource.com/csharp/RSPEC-3366)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L81
Make sure the use of 'this' doesn't expose partially-constructed instances of this class in multi-threaded environments. (https://rules.sonarsource.com/csharp/RSPEC-3366)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L92
The Cyclomatic Complexity of this method is 11 which is greater than 10 authorized. (https://rules.sonarsource.com/csharp/RSPEC-1541)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L249
Add the 'Async' suffix to the name of this method. (https://rules.sonarsource.com/csharp/RSPEC-4261)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L274
Add the 'Async' suffix to the name of this method. (https://rules.sonarsource.com/csharp/RSPEC-4261)
Build: COMETwebapp/Components/ParameterEditor/ParameterTable.razor#L30
Remove these redundant parentheses. (https://rules.sonarsource.com/csharp/RSPEC-3235)
Build: COMETwebapp/Components/ParameterEditor/ParameterTable.razor#L40
Replace this 'switch' statement with 'if' statements to increase readability. (https://rules.sonarsource.com/csharp/RSPEC-1301)
Build: COMETwebapp/Components/ParameterEditor/ParameterTable.razor#L40
Add a 'default' clause to this 'switch' statement. (https://rules.sonarsource.com/csharp/RSPEC-131)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L76
Make sure the use of 'this' doesn't expose partially-constructed instances of this class in multi-threaded environments. (https://rules.sonarsource.com/csharp/RSPEC-3366)
Build: COMETwebapp/ViewModels/Components/ParameterEditor/ParameterBaseRowViewModel.cs#L81
Make sure the use of 'this' doesn't expose partially-constructed instances of this class in multi-threaded environments. (https://rules.sonarsource.com/csharp/RSPEC-3366)