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.

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

Fix alignment with available commands #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Apr 13, 2024

  1. Fix alignment with available commands

    When upgrading the Color package from `v1.15.0` to `v1.16.0`, the list
    of available commands becomes incorrectly aligned:
    
    ```text
    Available Commands:
    
      completion Generate the autocompletion script for the specified shell
      help      Help about any command
      test      Test command for clarity
    ```
    
    This was related to an [issue][1] where reset codes were not correctly
    set. The [fix][2], which added extra reset codes, increased the length
    of the string (which includes the ANSI codes) causing the amount of
    padding to be insufficient.
    
    This change, which upgrades the Color package to `v1.16.0`, also
    increass the padding to account for the additional ANSI reset codes
    that are now part of the string. This restores the text alignment to
    how it was previously displayed.
    
    Unit tests also needed to be updated to account for this change and are
    passing successfully.
    
    The Color package had updated its dependencies which has caused the
    changes in `go.mod` to impact more than itself.
    
    Fixes: ivanpirog#4
    
    [1]: fatih/color#206
    [2]: fatih/color#210
    
    Signed-off-by: Michael Lorant <michael.lorant@nine.com.au>
    mikelorant committed Apr 13, 2024
    Configuration menu
    Copy the full SHA
    9287900 View commit details
    Browse the repository at this point in the history