Results 1044 comments of 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?