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
Move product selection to welcome #602
Conversation
b16006c
to
f186cf0
Compare
f186cf0
to
9476f25
Compare
return nil | ||
end | ||
setup_final_choice | ||
merge_and_run_workflow if selected_product |
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.
is this check really needed?
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.
No, it is not :)
end | ||
end | ||
|
||
# Most of the work is done by #retranslate_yast. If changing to english if |
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.
typo? is needed?
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, please just solve conflicts
test/inst_complex_welcome_test.rb
Outdated
@@ -48,6 +67,12 @@ | |||
|
|||
describe "#main" do | |||
let(:restarting) { false } |
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.
Please, remove also this from the test as it is not needed anymore
* master: Adapt unit test due changes in module version + changelog Skip inst_complex_welcome in auto mode.
09db101
to
4115a53
Compare
Rubocop is still unhappy |
Yes, you are right. I did not check Travis because I expected a dependencies problem (new yast2-packager and yast2 are not available yet). Now Rubocop is happy but it is failing due to missing dependencies :) |
Thnx ;) it LGTM |
Requires yast/yast-yast2#619, yast/yast-packager#273 and
yast/yast-country#138.
Move product's selection to welcome screen.