We think that at some point pytest added color codes to the summary output which broke these matcher regex. It looks like there is an issue about stripping color codes here: https://github.com/actions/runner/issues/2341 I've added wildcard (or no-space) elements instead of trying to match the color codes exactly (eg `\033\[31m` etc) because 1) that's not very readable 2) I was having trouble getting that to work with egrep. The goal is to match the target strings but not to match then later in the line as part of a test log or whatever. For the '= short test summary =' that should be pretty unique. For the ERROR|FAILED I reckon that could be a bit more common. I'm just matching with any mount of non-space characters because I reckon a space will crop up pretty early in any line where ERROR isn't the first work. I think this new matching will only apply to new or rebased PRs after it is landed. ref: https://github.com/qutebrowser/qutebrowser/issues/5390#issuecomment-1817503702 |
||
|---|---|---|
| .. | ||
| dev | ||
| testbrowser | ||
| __init__.py | ||
| asciidoc2html.py | ||
| cycle-inputs.js | ||
| dictcli.py | ||
| hist_importer.py | ||
| hostblock_blame.py | ||
| importer.py | ||
| keytester.py | ||
| link_pyqt.py | ||
| mkvenv.py | ||
| open_url_in_instance.sh | ||
| opengl_info.py | ||
| setupcommon.py | ||
| utils.py | ||