Rework Diagnostics Folding #140

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.

Sign up for GitHub

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

Merged

josephbirkner merged 5 commits into noserde from rework-diagnostics-folding

Mar 16, 2026

Merged

Rework Diagnostics Folding #140

diagnostics: Remove Environment & AST Dependencies

SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 13, 2026 in 1m 11s

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 183 in src/diagnostics.cpp

sonarqubecloud / SonarCloud Code Analysis

Extract this nested conditional operator into an independent statement.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZzaNojlHitx1-szXcQc&open=AZzaNojlHitx1-szXcQc&pullRequest=140

Check failure on line 62 in src/diagnostics.cpp

sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 32 to the 25 allowed.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZzd0Z4LtUTrntRZae3S&open=AZzd0Z4LtUTrntRZae3S&pullRequest=140

Check warning on line 153 in src/diagnostics.cpp

sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code.

See more on https://sonarcloud.io/project/issues?id=Klebert-Engineering_simfil&issues=AZzaNojlHitx1-szXcQf&open=AZzaNojlHitx1-szXcQf&pullRequest=140

View more details on SonarQubeCloud