Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Merge sp7 #1132

Merged
merged 12 commits into from
Dec 18, 2024
Merged

Merge sp7 #1132

merged 12 commits into from
Dec 18, 2024

Conversation

jreidinger
Copy link
Member

merge of #1129 and #1131

jreidinger and others added 12 commits December 17, 2024 13:56
@jreidinger
skip showing reboot message only if not explicitelly set on command l…
4c3af02 
…ine (bsc#1231522)
@jreidinger
changes
e439956
@lslezak
Self-update fix: remove the architecture suffix
fb63646 
from the version when reading the /.packages.root file from the inst-sys
@lslezak
Changes, version 4.6.14
c1246bc
@lslezak
Merge pull request #1130 from yast/self_update_fix_SP6
10b3723 
[SLE15-SP6] Fixed comparing versions in the self-update version check
@teclator
Update src/lib/installation/clients/inst_finish.rb
00b21eb 
Added extra line after guard clause.
@teclator
Update src/lib/installation/clients/inst_finish.rb
8405d71
@lslezak
Merge remote-tracking branch 'origin/SLE-15-SP6' into self_update_fix…
0c32a84 
…_SP7
@lslezak
Merge pull request #1131 from yast/self_update_fix_SP7
e95ed3f 
[SLE15-SP7] Fixed comparing versions in the self-update version check
@jreidinger
Merge remote-tracking branch 'origin/SLE-15-SP7' into show_dialog_exp…
bd5ffb4 
…licit
@jreidinger
Merge pull request #1129 from yast/show_dialog_explicit
d6e59e1 
Show dialog explicit
@jreidinger
Merge remote-tracking branch 'origin/SLE-15-SP7'
143f021
@jreidinger jreidinger mentioned this pull request Dec 17, 2024
Open
@coveralls
Copy link

Coverage Status

coverage: 40.928% (+0.006%) from 40.922%
when pulling 143f021 on merge_sp7
into b7f3c57 on master.

@jreidinger
Copy link
Member Author

@lslezak is it OK to apply also your changes to master? For my change it was required ( probably for TW testing )

@jreidinger jreidinger closed this Dec 17, 2024
@jreidinger jreidinger reopened this Dec 17, 2024
lslezak
lslezak approved these changes Dec 18, 2024
View reviewed changes
Copy link
Member

@lslezak lslezak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, my change should also go to master.

@jreidinger jreidinger merged commit 960c666 into master Dec 18, 2024
17 checks passed
@jreidinger jreidinger deleted the merge_sp7 branch December 18, 2024 08:43
@github-actions GitHub Actions
Copy link

✅ Autosubmission job #12389346528 successfully finished
✅ Created submit request #1231816

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lslezak lslezak lslezak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jreidinger @coveralls @lslezak @teclator