diff options
author | Eric Sunshine <sunshine@sunshineco.com> | 2021-12-13 09:30:54 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-12-14 01:15:29 +0300 |
commit | 5be30d0cd301f50e5f4dd992bc11ce75f457cf46 (patch) | |
tree | 61caace41b4929f5470b34d5767e7afa14544116 /t/chainlint/if-then-else.expect | |
parent | 0d7131763e7762dc34c407f32e3c63123001a292 (diff) |
chainlint.sed: drop subshell-closing ">" annotation
chainlint.sed inserts a ">" annotation at the beginning of a line to
signal that its heuristics have identified an end-of-subshell. This was
useful as a debugging aid during development of the script, but it has
no value to test writers and might even confuse them into thinking that
the linter is misbehaving by inserting line-noise into the shell code it
is validating. Moreover, its presence also potentially makes it
difficult to reuse the chainlint self-test "expect" output should a more
capable linter ever be developed. Therefore, drop the ">" annotation.
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/chainlint/if-then-else.expect')
-rw-r--r-- | t/chainlint/if-then-else.expect | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/t/chainlint/if-then-else.expect b/t/chainlint/if-then-else.expect index 3055d5606c..debcf7b756 100644 --- a/t/chainlint/if-then-else.expect +++ b/t/chainlint/if-then-else.expect @@ -11,10 +11,10 @@ cat fi ?!AMP?! echo poodle ->) && +) && ( if test -n ""; then echo very && echo empty fi ->) +) |