search

oamg / leapp-repository

Leapp repositories containing actors for the Leapp framework (https://github.com/oamg/leapp). Currently provides leapp repositories for in-place upgrades of RHEL systems.
Apache License 2.0
48 stars 144 forks source link

properly indent the list of supported OSes #1234

Closed evgeni closed 3 months ago

evgeni commented 3 months ago

before:

Risk Factor: high (inhibitor)
Title: The installed OS version is not supported for the in-place upgrade to the target RHEL version
Summary: The supported OS releases for the upgrade process:
 RHEL 8.8
RHEL 8.10
RHEL-SAPHANA 8.8
RHEL-SAPHANA 8.10

after:

Risk Factor: high (inhibitor)
Title: The installed OS version is not supported for the in-place upgrade to the target RHEL version
Summary: The supported OS releases for the upgrade process:
    - RHEL 8.8
    - RHEL 8.10
    - RHEL-SAPHANA 8.8
    - RHEL-SAPHANA 8.10
pirat89 commented 3 months ago

@evgeni I think it would be even better to used FMT_LIST_SEPARATOR = '\n - ' defined in many other actors, to have the list formatted in the same way as anywhere else. I am ok to go with this intendantion fix now but it would be pity to not change it already. wdyt? in case you want to change it :)

evgeni commented 3 months ago

Like this? Or less indentation? Wasn't sure :)

pirat89 commented 3 months ago

@evgeni nice PR number :-D