Skip to content

docs: add fresh SonarCloud reanalysis report for PR 3540#67

Closed
Easton97-Jens wants to merge 2 commits intov3/master-json2from
codex/perform-complete-analysis-of-pr-3540
Closed

docs: add fresh SonarCloud reanalysis report for PR 3540#67
Easton97-Jens wants to merge 2 commits intov3/master-json2from
codex/perform-complete-analysis-of-pr-3540

Conversation

@Easton97-Jens
Copy link
Copy Markdown
Owner

@Easton97-Jens Easton97-Jens commented Apr 13, 2026

Motivation

  • Provide an up-to-date SonarCloud reanalysis forso reviewers have a live Quality Gate, measures and the full issue inventory; analysis is based on commit bbde416a92212f1fc4a975361e73a3b07356a526.

Description

  • Add reports/pr3540_sonar_reanalyse.md, a generated Markdown report that includes the Quality Gate status and measures, a full table of all Sonar issues (207) plus a gate entry, a filtered table of open findings (52), a JSON-backend-focused table (per-file open issues and minimal fixes), aggregated counts by severity/status, and explicit analysis limitations.

Testing

  • SonarCloud and GitHub API calls used to build the report were executed successfully for pullRequest=3540 and the report file was written locally as reports/pr3540_sonar_reanalyse.md.
  • Existing PR check runs for the HEAD commit show SonarCloud Code Analysis and multiple GitHub Actions build jobs in success state for the analyzed commit.

Codex Task

Easton97-Jens and others added 2 commits April 13, 2026 18:28
sonarqubecloud error corrected

Refactor JSONAdapter parse method to use parseImpl

Refactor JSONAdapter parse methods to remove adapter parameter

Add parseImpl method to json_adapter.h

Refactor JSON field handling in regression tests

Refactor JSON field processing by introducing a reusable for_each_json_field function. This change simplifies the code in multiple update functions by eliminating redundant logic.

Refactor parsing functions for improved validation

Cleanup: Remove unused parsing functions

Removed unused functions related to parsing and scenario building in json_benchmark.cc.

Improve benchmark output and add bilingual benchmark docs

Apply minimal Sonar-focused fixes in benchmark parser/script

Measure benchmark timer only around transaction loop
@Easton97-Jens Easton97-Jens deleted the codex/perform-complete-analysis-of-pr-3540 branch April 13, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant