Skip to content

Remove ERROR from parse_rules#78

Merged
FreddieAkeroyd merged 2 commits into
masterfrom
rerpha-patch-1
May 5, 2026
Merged

Remove ERROR from parse_rules#78
FreddieAkeroyd merged 2 commits into
masterfrom
rerpha-patch-1

Conversation

@rerpha

@rerpha rerpha commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

ie. https://epics-jenkins.isis.rl.ac.uk/job/InstrumentChecker/3447/parsed_console/

where FAILED is the only actual useful information and the process returns 1 anyway if it falls over.

@rerpha rerpha added this to Tasks Apr 29, 2026
@rerpha rerpha moved this to Flash Review in Tasks Apr 29, 2026
@FreddieAkeroyd

Copy link
Copy Markdown
Member

Might it be worth matching FAIL [ instead of FAILED so we get a link to each failure?

@FreddieAkeroyd
FreddieAkeroyd self-requested a review April 30, 2026 14:46
@FreddieAkeroyd
FreddieAkeroyd merged commit 0ceac21 into master May 5, 2026
4 checks passed
@FreddieAkeroyd
FreddieAkeroyd deleted the rerpha-patch-1 branch May 5, 2026 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants