-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Open
Description
runner/src/Runner.Worker/IssueMatcher.cs
Line 529 in 9426c35
| throw new ArgumentException($"The property '{propertyName}' is set to {newValue} which is out of range"); |
Test case:
check-spelling-sandbox/upgraded-garbanzo-vigilant-journey@91674af
Results:
https://github.com/check-spelling-sandbox/upgraded-garbanzo-vigilant-journey/actions/runs/22208910261
Annotations
2 errors
test
The property 'line' is set to 2 which is out of range
test
Unable to process command '::add-matcher::/home/runner/work/upgraded-garbanzo-vigilant-journey/upgraded-garbanzo-vigilant-journey/a.json' successfully.
Notes:
- It'd be helpful if the error message reported a file name and line number (preferably as an annotation, this happens for unhappy workflows)
- It'd be helpful if the problem matcher complained that the
regexpfield was not present - If you're going to complain about fields, it's best to complain in numerical sort order (
1before2) instead of alphabetical key order (linebeforemessage) - I don't understand why failure to process a problem matcher is deemed as fatal. Problem matchers are merely aids to help people interpret logs, they aren't mission critical. What's critical is that a workflow run to completion.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels