Commit graph

4 commits

Author SHA1 Message Date
mehrdadn
b14728d999
Shellcheck quoting (#9596)
* Fix SC2006: Use $(...) notation instead of legacy backticked `...`.

* Fix SC2016: Expressions don't expand in single quotes, use double quotes for that.

* Fix SC2046: Quote this to prevent word splitting.

* Fix SC2053: Quote the right-hand side of == in [[ ]] to prevent glob matching.

* Fix SC2068: Double quote array expansions to avoid re-splitting elements.

* Fix SC2086: Double quote to prevent globbing and word splitting.

* Fix SC2102: Ranges can only match single chars (mentioned due to duplicates).

* Fix SC2140: Word is of the form "A"B"C" (B indicated). Did you mean "ABC" or "A\"B\"C"?

* Fix SC2145: Argument mixes string and array. Use * or separate argument.

* Fix SC2209: warning: Use var=$(command) to assign output (or quote to assign string).

Co-authored-by: Mehrdad <noreply@github.com>
2020-07-21 21:56:41 -05:00
Sven Mika
676a011784
Make sure pre-push is executable. (#7079) 2020-02-07 11:38:14 -08:00
Eric Liang
1a8fa5d2fa
Clean up top level Ray dir (#5404) 2019-08-08 23:35:55 -07:00
Edward Oakes
2b7b7c7547 Add linting pre-push hook (#5154) 2019-07-09 21:49:12 -07:00