-
Notifications
You must be signed in to change notification settings - Fork 357
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
Add GCC compiler problem matcher to CI #2935
Conversation
@jougs @terhorstd I think you had already looked at this and expressed support. Could you give formal feedback? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a strong opinion on including this in the CI. The GitHub parsing of stdout magic strings is very irritating, but the changes don't constitute a strong binding to this proprietary service.
Just a comment: I feel like this is useful. One could may be use this action from the marketplace. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Resolves #2779.
This PR introduces a GitHub Actions problem matcher which scans the GCC compiler output for a specified regex pattern and surface the information in the UI as annotations.
The problem matcher is identical to the one used in VS Code's
cpptools
:https://github.com/microsoft/vscode-cpptools/blob/2cc55ea6ebe96b91c118c84784e980bdfaf20304/Extension/package.json#L303-L320.