openfaas / golang-http-template

Golang templates for OpenFaaS using HTTP extensions
https://www.openfaas.com/
MIT License
106 stars 57 forks source link

fix: update modules-cleanup.sh #84

Closed mrwormhole closed 2 years ago

mrwormhole commented 2 years ago

Signed-off-by: Talha Altınel talhaaltinel@hotmail.com

Description

ditching out sh: out of index message from consoles

How Has This Been Tested?

I have tested on my own by doing

faas-cli template pull https://github.com/openfaas/golang-http-template faas-cli new --lang golang-http cow-fn faas-cli build -f cow-fn.yml

How are existing users impacted? What migration steps/scripts do we need?

nothing big deal, just don't like red messages

Checklist:

I have:

alexellis commented 2 years ago

Thanks for looking into this.

What was wrong and how does adding that line fix it?

mrwormhole commented 2 years ago

posix shell issue, if a variable is unset and empty string, it can not evaluate/interpret if [ "$DEBUG" -eq 1 ]; then