Skip to content
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

Text report adds rules from information issues to the number of violated rules #162

Closed
andreaskern74 opened this issue Sep 6, 2024 · 0 comments · Fixed by #167
Closed
Labels
isType:Bug An issue that contains contradictions or errors in the standard.

Comments

@andreaskern74
Copy link
Collaborator

Describe the bug

The text report contains a number of violated rules at the end of the report. This is a list of all addressed rule, which can be found in the reported issues.

violated_rules.insert((*it_Issue)->GetRuleUID());

This will also add informational thing to the list.

Expected behavior

Would be nice to have three lists here.

If this is currently too much effort, then probably add only rules for warnings and errors to the list or rename the variable and the output in the text report.

@andreaskern74 andreaskern74 added isType:Bug An issue that contains contradictions or errors in the standard. isState:New A new issue that needs to be classified to a type. labels Sep 6, 2024
@hoangtungdinh hoangtungdinh linked a pull request Sep 10, 2024 that will close this issue
@hoangtungdinh hoangtungdinh moved this to Can close in Quality Checker Sep 10, 2024
@andreaskern74 andreaskern74 removed the isState:New A new issue that needs to be classified to a type. label Sep 12, 2024
@hoangtungdinh hoangtungdinh moved this from Can close to Done in Quality Checker Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
isType:Bug An issue that contains contradictions or errors in the standard.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant