New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add ability for WSL desktop file whitelisting #1020
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, although I am not that familiarized with this part of the code.
library/desktop/test/desktop_test.rb
Outdated
@@ -34,9 +34,58 @@ | |||
"add-on" => { "Name" => "YaST Add-On Products" }, | |||
"lan" => { "Name" => "YaST Network" }, | |||
"services-manager" => { "Name" => "YaST Services Manager" }, | |||
"sw-single" => { "Name"=>"YaST Software Management" }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NP: missing spaces around =>
Ups, there are conflicts. Please, fix them ;) |
@joseivanlopez fixed, I worry no one is familiar with this piece of code as it is ncurses only control center + desktop parses which is also not used often. |
✔️ Public Jenkins job #181 successfully finished |
✔️ Internal Jenkins job #79 successfully finished |
reason: WSL is limited environment, so many modules does not make sense to show. So whitelisting only ones that has specific key in desktop file.