Skip to content

Hotkey picker

Hotkey picker #527

Triggered via pull request January 13, 2025 10:49
Status Success
Total duration 11m 47s
Artifacts 1

build.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

24 warnings and 9 notices
build
Your workflow is using a version of actions/cache that is scheduled for deprecation, actions/cache@v1. Please update your workflow to use either v3 or v4 of actions/cache to avoid interruptions. Learn more: https://github.blog/changelog/2024-12-05-notice-of-upcoming-releases-and-breaking-changes-for-github-actions/#actions-cache-v1-v2-and-actions-toolkit-cache-package-closing-down
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L95
Method `OnPreviewMouseDown` coverage is below the threshold 50%
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L45
Method `set_HotkeyCompletedCommand` coverage is below the threshold 50%
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L45
Method `get_HotkeyCompletedCommand` coverage is below the threshold 50%
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L31
Class `Generic.HotkeyPicker` coverage is below the threshold 50%
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L50
Method `OnPreviewKeyDown` coverage is below the threshold 50%
Check .NET source code coverage: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L119
Method `OnPreviewLostKeyboardFocus` coverage is below the threshold 50%
Redundant member initializer: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L47
Initializing field by default value is redundant
Redundant class or interface specification in base types list: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L31
Base type 'UserControl' is already specified in other parts
Redundant using directive: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L18
Using directive is not required by the code and can be safely removed
Redundant using directive: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L19
Using directive is not required by the code and can be safely removed
Redundant using directive: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L21
Using directive is not required by the code and can be safely removed
Unused local variable: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L89
Local variable 'converter' is never used
Redundant namespace alias: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml#L24
Namespace alias 'materialDesign' is not required by code and can be safely removed
Redundant namespace alias: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml#L25
Namespace alias 'validation' is not required by code and can be safely removed
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L50
Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L54
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L89
Remove the unused local variable 'converter'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L50
Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L54
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
build: src/soulsplitter/UI/Generic/HotkeyPicker.xaml.cs#L89
Remove the unused local variable 'converter'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
build: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L50
Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
build: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L54
Merge this if statement with the enclosing one. (https://rules.sonarsource.com/csharp/RSPEC-1066)
build: src/SoulSplitter/UI/Generic/HotkeyPicker.xaml.cs#L89
Remove the unused local variable 'converter'. (https://rules.sonarsource.com/csharp/RSPEC-1481)
Redundant lambda signature parentheses: src/Cli/Program.cs#L48
Redundant lambda signature parentheses

Artifacts

Produced during runtime
Name Size
SoulSplitter
9.05 MB