comaniac opened a new pull request #6469: URL: https://github.com/apache/incubator-tvm/pull/6469
This PR fixes some issues I encountered when running `git-black.sh` on `bash`: 1. A syntax error (an unexpected ")") in a message. This seems fine on `zsh` but causes crash on `bash`. 2. A guard to prevent failure when no changes in Python file (`$FILES` would be an unbound variable). 3. Remove `HEAD` in the command of listing changed Python files so that it will also cover upstaged changes. While the first two points may fail the CI, the third point is optional. I just feel that it's more convenient for people to format their changes before making a commit. I could revert this one if you folks don't prefer to have it. cc @jroesch @tqchen ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
