diff options
author | Aneesh Agrawal <aneeshusa@gmail.com> | 2016-08-04 18:12:54 -0400 |
---|---|---|
committer | Aneesh Agrawal <aneeshusa@gmail.com> | 2016-08-05 09:29:10 -0400 |
commit | 9231ca1c6991c2eaf0d7e08674b2fc9c84a31695 (patch) | |
tree | 15c0c7cfbc998342d053e259d9138b6003a170bc /python/tidy/servo_tidy_tests/test_tidy.py | |
parent | 79ef9b4efc307aa0aebfbeb5b0c120a3ad525399 (diff) | |
download | servo-9231ca1c6991c2eaf0d7e08674b2fc9c84a31695.tar.gz servo-9231ca1c6991c2eaf0d7e08674b2fc9c84a31695.zip |
Add lint to ensure substitutions use the full form
Check that any variable substitutions use the full ${VAR} form,
not just $VAR (but don't check for quoting yet).
Diffstat (limited to 'python/tidy/servo_tidy_tests/test_tidy.py')
-rw-r--r-- | python/tidy/servo_tidy_tests/test_tidy.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/python/tidy/servo_tidy_tests/test_tidy.py b/python/tidy/servo_tidy_tests/test_tidy.py index 58e8f6e5fd3..2eee04a19a3 100644 --- a/python/tidy/servo_tidy_tests/test_tidy.py +++ b/python/tidy/servo_tidy_tests/test_tidy.py @@ -53,6 +53,7 @@ class CheckTidiness(unittest.TestCase): self.assertEqual('script does not have shebang "#!/usr/bin/env bash"', errors.next()[2]) self.assertEqual('script is missing options "set -o errexit", "set -o pipefail"', errors.next()[2]) self.assertEqual('script should not use backticks for command substitution', errors.next()[2]) + self.assertEqual('variable substitutions should use the full \"${VAR}\" form', errors.next()[2]) self.assertNoMoreErrors(errors) def test_rust(self): |