Skip to content

New feature to report unknown rules / warnings#2806

Merged
guwirth merged 4 commits intoSonarOpenCommunity:masterfrom
guwirth:unknown-rules-warning
Nov 14, 2024
Merged

New feature to report unknown rules / warnings#2806
guwirth merged 4 commits intoSonarOpenCommunity:masterfrom
guwirth:unknown-rules-warning

Conversation

@guwirth
Copy link
Copy Markdown
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 force-pushed the unknown-rules-warning branch from ef6a3f5 to fb4e5ec Compare November 14, 2024 14:43
@guwirth guwirth merged commit f155c37 into SonarOpenCommunity:master Nov 14, 2024
@guwirth guwirth deleted the unknown-rules-warning branch February 17, 2026 10:19
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