-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Problem Matcher for testing/CI #101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
We've gone back and forth on this. We considered adding problem matchers directly for mocha, jest, eslint etc. but that's a growing list and it would be hard to remove. We also discussed creating other setup actions like setup-jest etc. and also inputs to this but that's cumbersome to the consumers in their yaml. I think the best direction would be if this action detected a framework (jest is in the package.json etc. etc.) and dynamically registered the problem matcher. There's a I'm also about to start a v2-beta so sketching this out could be part of that. |
@bryanmacfarlane Thanks for the reply, I agree this is a good idea, using It does look like FYI on relevant thread over at mocha: mochajs/mocha#4132 It doesn't seem like the maintainers of mocha are particularly interested in a dedicated reporter, so we'll either need to create a new reporter or conform with one of their built-in reporters to support this. I'm very happy to contribute to the matcher here since we use Actions pretty heavily and the current scroll all the way to bottom for test case errors is slightly inconvenient 😅 |
Version bump 1.6.0
How can we enable problem matchers for various testing tools like https://github.com/mochajs/mocha? Is there a reporting format that we should use for it to be matched by the runner when errors occur in the test cases? I saw the problem matchers for
eslint
likeestlint-stylish
but didn't see anything else.If anyone can point me in the right direction, I'm happy to create the matcher if it doesn't already exist.
The text was updated successfully, but these errors were encountered: