Skip to content

Suppress SonarCloud cpp:S5145 and cpp:S3659 false positives#251

Merged
Bronek merged 1 commit into
mainfrom
bronek/sonarqube_tain_removal
Jun 30, 2026
Merged

Suppress SonarCloud cpp:S5145 and cpp:S3659 false positives#251
Bronek merged 1 commit into
mainfrom
bronek/sonarqube_tain_removal

Conversation

@Bronek

@Bronek Bronek commented Jun 30, 2026

Copy link
Copy Markdown
Member

No description provided.

…lse positives

Assisted-by: Claude:claude-opus-4-8
@Bronek Bronek merged commit 67332f9 into main Jun 30, 2026
2 checks passed
@augmentcode

augmentcode Bot commented Jun 30, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: Updates SonarCloud project configuration to suppress two specific C++ rule false-positives.

Changes:

  • Ignores cpp:S5145 under examples/**/* where echoing user input is intentional in demos
  • Ignores cpp:S3659 project-wide to allow alternative operator tokens (e.g. not) as a deliberate readability choice

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant