search

weaveworks / launcher

Weave Cloud Launcher
Apache License 2.0
10 stars 13 forks source link

agent: Fix wcPollUrl resolve bug #102

Closed aaron7 closed 6 years ago

rndstr commented 6 years ago

out of curiosity, why the need to switch to bash?

aaron7 commented 6 years ago

out of curiosity, why the need to switch to bash?

We had sh silently fail on unsupported syntax like if [[ ]] (double brackets) and we feel more comfortable writing in bash which means hopefully less will go wrong. It's likely we will switch to another language as the integration test suite grows.