Skip to content

Conversation

@mrwormhole
Copy link
Contributor

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:

  • updated the documentation and/or roadmap (if required)
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

Signed-off-by: Talha Altınel <talhaaltinel@hotmail.com>
@alexellis
Copy link
Member

Thanks for looking into this.

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

@mrwormhole
Copy link
Contributor Author

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

@alexellis alexellis merged commit 1a83e7f into openfaas:master Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants