Skip to content

Commit c7261ca

Browse files
committed
github: Fix internal problem matcher for color ASCII escape codes
The internal warnings/errors problem matcher regex still does not highlight the messages properly, even though I tested the regex. Unfortunately, the reason is only obvious when looking at the raw GitHub logs, which shows the raw color ASCII escape codes that print_red() and print_yellow() use. Add the escape codes to the regex to fix the matching, which ends up making the matcher more robust because it will only highlight the color messages that the script generates. Signed-off-by: Nathan Chancellor <[email protected]>
1 parent 1172443 commit c7261ca

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/problem-matchers/internal.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"pattern": [
77
{
88
"__comment_regexp1": "ERROR: Build is not finished on TuxSuite's side!",
9-
"regexp": "^(WARNING|ERROR):\\s+(.*)$",
9+
"regexp": "^\\x1b\\[[0-9]+m(WARNING|ERROR):\\s+(.*)\\x1b\\[0m$",
1010
"severity": 1,
1111
"message": 2
1212
}

0 commit comments

Comments
 (0)