openfaas/faasd

Using $SUDO instead of sudo in install.sh

yoshino-s opened this issue · 1 comments

yoshino-s commented

Due diligence

Why do you need this?

We use $SUDO in other place, but why we use sudo only in here?

Expected Behaviour

install_arkade(){
  curl -sLS https://get.arkade.dev | $SUDO sh
  arkade --help
}

Current Behaviour

install_arkade(){
  curl -sLS https://get.arkade.dev | sudo sh
  arkade --help
}

Are you a GitHub Sponsor (Yes/No?)

Check at: https://github.com/sponsors/openfaas

  • Yes
  • No

List All Possible Solutions and Workarounds

Just replace it will be fine.

alexellis commented

Thanks for letting us know.