Randy Lai
Randy Lai
AFAIK, the header is only used when the file is an executable. I don't think it is the cause of #159.
`#! /usr/bin/env bash` should just work ``` (penguin)-~$ cat ./foo.sh #! /usr/bin/env bash if [[ true ]]; then echo TRUE else echo FALSE fi (penguin)-~$ sh ./foo.sh ./foo.sh: 3: ./foo.sh:...
Thank you for working on it. I wanted to rewrite the script for a long time. We use a different file `run_tests.ps1` for windows. That one needs to be patched...
Actually, the windows support was mainly contributed by @guillermooo. I personally do not have any preference. Actually, it may be easier for a maintenance point of view if any platform...
Sure. The whole script was not well written indeed.
Thank for the initiative work.
Sure. As a note, we will also need to modify the bootstrap code for the CI services so `sublime_lib` is installed automatically.
Installation of Package Control is optional. If a user didn't install Package Control manually via `bootstrap install_package_control`, the script will download the current only dependency `coverage` from the github repo.
I used Travis docker machine for testing.
It seems to a dirty fix. I would rather check the source if the error: why coverage was not installed or detected?