Skip to content

[feat] Display guideline levels in the guideline statistics table#4787

Merged
bruntib merged 1 commit intoEricsson:masterfrom
bruntib:guideline_level
Mar 31, 2026
Merged

[feat] Display guideline levels in the guideline statistics table#4787
bruntib merged 1 commit intoEricsson:masterfrom
bruntib:guideline_level

Conversation

@bruntib
Copy link
Copy Markdown
Contributor

@bruntib bruntib commented Feb 24, 2026

Some guidelines have their own severity ranking, like the levels at SEI-CERT. In this commit these are also displayed at the guideline statistics table.

@bruntib bruntib added this to the release 6.28.0 milestone Feb 24, 2026
@bruntib bruntib marked this pull request as ready for review March 24, 2026 22:26
@bruntib bruntib requested a review from vodorok as a code owner March 24, 2026 22:26
Comment thread web/api/report_server.thrift Outdated
1: string ruleId, // The identifier of the rule.
2: string title, // The rule summary.
3: string url, // The link of the rule page.
4: list<string> checkers, // List of checker names,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
4: list<string> checkers, // List of checker names,
4: list<string> checkers, // List of checker names.

Comment thread web/api/report_server.thrift Outdated
Copy link
Copy Markdown
Collaborator

@gulyasgergely902 gulyasgergely902 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If not required in the next release, please, wait with this until the migration is complete to Vue 3.

Some guidelines have their own severity ranking, like the levels at
SEI-CERT. In this commit these are also displayed at the guideline
statistics table.
@bruntib bruntib requested a review from barnabasdomozi March 29, 2026 22:58
@bruntib
Copy link
Copy Markdown
Contributor Author

bruntib commented Mar 29, 2026

@gulyasgergely902, Sorry, this will be needed :(

Copy link
Copy Markdown
Collaborator

@barnabasdomozi barnabasdomozi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bruntib bruntib merged commit 93996c6 into Ericsson:master Mar 31, 2026
9 checks passed
@bruntib bruntib deleted the guideline_level branch April 24, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants