From b7112e02db56c611f5ab4efddea4362e4af5c5bf Mon Sep 17 00:00:00 2001 From: Eugene Zhukov Date: Tue, 17 Nov 2020 10:02:06 +0200 Subject: [PATCH] Update curl arguments in installation.md Double -s argument does not make sense according to curl manual page. -s stands for silent, while -S (capital S) stands for showing the error. When used in combination, curl shows an error message if it fails, but disables progress meter. Finally, in the end of sh command there is -d, which stands for debug and contradicts -sS curl arguments. I suggest to remove curl silencers all together, because more debug is better in CIs. I also suggest to use --location instead of -L for clarity. --- docs/installation.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/installation.md b/docs/installation.md index 076643fd..a139f694 100644 --- a/docs/installation.md +++ b/docs/installation.md @@ -74,11 +74,11 @@ easily generating this script. ```bash # For Default Installion to ./bin with debug logging -sh -c "$(curl -ssL https://taskfile.dev/install.sh)" -- -d +sh -c "$(curl --location https://taskfile.dev/install.sh)" -- -d # For Installation To /usr/local/bin for userwide access with debug logging # May require sudo sh -sh -c "$(curl -ssL https://taskfile.dev/install.sh)" -- -d -b /usr/local/bin +sh -c "$(curl --location https://taskfile.dev/install.sh)" -- -d -b /usr/local/bin ```