Skip to content

Commit 301133a

Browse files
committed
Misc: Fixup for accept-expected-changes-from-ci.py
Allow empty lines while looking for diff (between `---expected` and `FAILED`). This can happen when there is `Locations outside the test directory` since an empty line is printed after that warning message (this output can be interleaved with the diff due to parallel execution).
1 parent 284ca5e commit 301133a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

misc/scripts/accept-expected-changes-from-ci.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ def make_patches_from_log_file(log_file_lines) -> List[Patch]:
114114
while True:
115115
next_line = parse_log_line(next(lines))
116116
# it can be the case that
117-
if next_line[0] in (" ", "-", "+", "@"):
117+
if next_line and next_line[0] in (" ", "-", "+", "@"):
118118
lines_changed.append(next_line)
119119
if "FAILED" in next_line:
120120
break

0 commit comments

Comments
 (0)