Skip to content

Add support for CodeChecker warning/error reasons#146

Merged
MiklosMagyari merged 3 commits intoEricsson:mainfrom
Discookie:ericsson/not-analyzed
Jul 15, 2024
Merged

Add support for CodeChecker warning/error reasons#146
MiklosMagyari merged 3 commits intoEricsson:mainfrom
Discookie:ericsson/not-analyzed

Conversation

@Discookie
Copy link
Copy Markdown
Collaborator

Fixes (partially) #137

CodeChecker is assumed to output a summary of its analysis in a user-readable manner, on its last stderr line.

This reason is now displayed on the sidebar, if available.
Also added support for differentiating between CodeChecker warnings/errors.

image

(The full reason is of course shown on hover.)

@Discookie Discookie added the enhancement New feature or request label May 2, 2024
@Discookie Discookie requested a review from vodorok May 2, 2024 08:28
@Discookie
Copy link
Copy Markdown
Collaborator Author

Reference for the last-line assumption on the CodeChecker side: Ericsson/codechecker#4222

@dkrupp dkrupp requested a review from MiklosMagyari July 3, 2024 10:52
Comment thread src/backend/executor/process.ts
Comment thread src/editor/notifications.ts
@Discookie Discookie force-pushed the ericsson/not-analyzed branch from cdd1df8 to e3c1391 Compare July 12, 2024 12:47
@MiklosMagyari MiklosMagyari merged commit 5374a75 into Ericsson:main Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants