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

Fix/explain violations of unrelated guidelines #199

Merged
merged 63 commits into from
Sep 1, 2023

Conversation

PhilippSalvisberg
Copy link
Collaborator

Fixes #179, fixes #181, fixes #183, fixes #184, fixes #186, fixes #187, fixes #188, #fixes 189, fixes #190, fixes #191, fixes #192, fixes #193, fixes 194

The idea is to have only violations in the bad examples for exactly one guideline. However, this cannot always be achieved. In those cases, a NOSONAR marker should be applied or a normal comment added (when the NOSONAR marker would hide the wanted violation in the bad example).

@sonarcloud
Copy link

sonarcloud bot commented Sep 1, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability C 2 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 15 Code Smells

0.0% 0.0% Coverage
1.1% 1.1% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@PhilippSalvisberg PhilippSalvisberg merged commit 1dc334b into main Sep 1, 2023
2 of 3 checks passed
@PhilippSalvisberg PhilippSalvisberg deleted the bugfix/violations-in-code-blocks branch September 13, 2023 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment