alexellis / arkade

Open Source Marketplace For Developer Tools
https://blog.alexellis.io/kubernetes-marketplace-two-year-update/
MIT License
4.24k stars 287 forks source link

Print a new line after downloading apps #530

Closed alexellis closed 3 years ago

alexellis commented 3 years ago

Expected Behaviour

A new line should be printed

Current Behaviour

It isn't

alex@alex-nuc8:~/go/src/github.com/alexellis/arkade$ go build && ./arkade get faas-cli
Downloading: faas-cli
2021/10/12 11:28:05 Looking up version for faas-cli
2021/10/12 11:28:05 Found: 0.13.13
Downloading: https://github.com/openfaas/faas-cli/releases/download/0.13.13/faas-cli
8.04 MiB / 8.04 MiB [------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00%
/tmp/faas-cli written.
2021/10/12 11:28:06 Looking up version for faas-cli
2021/10/12 11:28:06 Found: 0.13.13
2021/10/12 11:28:06 Copying /tmp/faas-cli to /home/alex/.arkade/bin/faas-cli

Tool written to: /home/alex/.arkade/bin/faas-cli

# Add arkade binary directory to your PATH variable
export PATH=$PATH:$HOME/.arkade/bin/

# Test the binary:
/home/alex/.arkade/bin/faas-cli

# Or install with:
sudo mv /home/alex/.arkade/bin/faas-cli /usr/local/bin/alex@alex-nuc8:~/go/src/github.com/alexellis/arkade$ 

Possible Solution

Add a newline before printing the message, or switch to fmt.Println(), Printf("%s",msg) is redundant

Steps to Reproduce (for bugs)

  1. Listed above

Context

See #516

alexellis commented 3 years ago

@yankeexe PTAL