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

New feature to report unknown rules / warnings #2806

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

guwirth
Copy link
Collaborator

@guwirth guwirth commented Nov 11, 2024

Starting with cxx plugin version 2.2 all CXX repositories contain a rule with the key unknown. By activating this rule, unknown rules in the reports are not discarded but mapped to this rule. This helps to identify unknown rules, e.g. from newer versions of a tool.


This change is Reviewable

jenspopp and others added 3 commits January 8, 2024 12:39
Increase count for rules for CLANGTIDY, CPPCHECK and GCC to account
for new "unknown" rule
Increase count of possible properties to account for
sonar.cxx.unknown.rule.id
- merge remote-tracking branch 'remotes/jenspopp/master' into unknown-rules-warning (rebase)
@guwirth guwirth added this to the 2.2.0 milestone Nov 11, 2024
@guwirth guwirth self-assigned this Nov 11, 2024
@guwirth guwirth force-pushed the unknown-rules-warning branch 4 times, most recently from 319128f to 7ad0026 Compare November 14, 2024 13:28
…SONAR_TOKEN instead

- remove 'sonar.cxx.unknown.rule.id' property
- add unknown to all repositories
- activate mapping only if 'unknown' rule is active
- add integration tests
- correct hashValue and equal of CxxReportLocation
@guwirth guwirth merged commit f155c37 into SonarOpenCommunity:master Nov 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Rule template for GCC custom rules not working due to key validation: - sign not allowed
2 participants