Merge pull request #10216 from nobe4/improve-padded-display
fix: padded display
This commit is contained in:
commit
a6ea8fe4ed
1 changed files with 6 additions and 9 deletions
|
|
@ -11,6 +11,7 @@ import (
|
||||||
"path"
|
"path"
|
||||||
"path/filepath"
|
"path/filepath"
|
||||||
"runtime"
|
"runtime"
|
||||||
|
"slices"
|
||||||
"strings"
|
"strings"
|
||||||
"sync"
|
"sync"
|
||||||
|
|
||||||
|
|
@ -469,18 +470,14 @@ func (m *Manager) Upgrade(name string, force bool) error {
|
||||||
}
|
}
|
||||||
|
|
||||||
func (m *Manager) upgradeExtensions(exts []*Extension, force bool) error {
|
func (m *Manager) upgradeExtensions(exts []*Extension, force bool) error {
|
||||||
var longestExtName = 0
|
var longestExt = slices.MaxFunc(exts, func(a, b *Extension) int {
|
||||||
for _, ext := range exts {
|
return len(a.Name()) - len(b.Name())
|
||||||
l := len(ext.Name())
|
})
|
||||||
if len(ext.Name()) > longestExtName {
|
var longestExtName = len(longestExt.Name())
|
||||||
longestExtName = l
|
|
||||||
}
|
|
||||||
}
|
|
||||||
format := fmt.Sprintf("[%%%ds]: ", longestExtName)
|
|
||||||
|
|
||||||
var failed bool
|
var failed bool
|
||||||
for _, f := range exts {
|
for _, f := range exts {
|
||||||
fmt.Fprintf(m.io.Out, format, f.Name())
|
fmt.Fprintf(m.io.Out, "[%*s]: ", longestExtName, f.Name())
|
||||||
currentVersion := displayExtensionVersion(f, f.CurrentVersion())
|
currentVersion := displayExtensionVersion(f, f.CurrentVersion())
|
||||||
err := m.upgradeExtension(f, force)
|
err := m.upgradeExtension(f, force)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue