Skip to content

Fix/irrelevant buffer owning dataset #1340

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
figueroa1395 merged 15 commits into from
Mar 26, 2026
+249 −54

NOLINT per line

76ef51b
Select commit
Loading
Failed to load commit list.
Merged

Fix/irrelevant buffer owning dataset #1340

NOLINT per line
76ef51b
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 26, 2026 in 2m 9s

Quality Gate passed

Issues
2 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 103 in tests/native_api_tests/test_api_dataset.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the commented out code. 

See more on https://sonarcloud.io/project/issues?id=PowerGridModel_power-grid-model&issues=AZ0pW_7q2BMkbzG0Hx0E&open=AZ0pW_7q2BMkbzG0Hx0E&pullRequest=1340

Check warning on line 350 in power_grid_model_c/power_grid_model_cpp/include/power_grid_model_cpp/dataset.hpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "contains" member function. 

See more on https://sonarcloud.io/project/issues?id=PowerGridModel_power-grid-model&issues=AZ0a7IqTbU7bKjPpF3HM&open=AZ0a7IqTbU7bKjPpF3HM&pullRequest=1340
View more details on SonarQubeCloud