Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix(autocomplete): do not show update message on autocomplete
Browse files Browse the repository at this point in the history
curzolapierre committed Feb 19, 2024
1 parent 3d0f975 commit f31b720
Showing 2 changed files with 10 additions and 5 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -2,6 +2,8 @@

### To be Released

* fix(autocomplete): do not show update message on autocomplete

### 1.30.1

* feat(addons): add aliases for database user management (i.e. using format: `database-users-<action>`) ([PR#1031](https://github.com/Scalingo/cli/pull/1031))
13 changes: 8 additions & 5 deletions scalingo/main.go
Original file line number Diff line number Diff line change
@@ -139,11 +139,14 @@ func main() {
fmt.Println("Fail to run command:", err)
}

// We want to display to the user if a new version is available
// Whatever the success of the execution of their command is.
updateCheckErr := update.Check()
if updateCheckErr != nil {
debug.Println("Failed to check if executable should be updated", updateCheckErr)
// Do not show update check during autocomplete
if !bashComplete {
// We want to display to the user if a new version is available
// Whatever the success of the execution of their command is.
updateCheckErr := update.Check()
if updateCheckErr != nil {
debug.Println("Failed to check if executable should be updated", updateCheckErr)
}
}

if err != nil {

0 comments on commit f31b720

Please sign in to comment.