Skip to content

Add missing Python datasource bindings #164

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 1 commit into from
May 8, 2026

Add missing Python datasource bindings

70293c2
Select commit
Loading
Failed to load commit list.
Merged

Add missing Python datasource bindings #164

Add missing Python datasource bindings
70293c2
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 8, 2026 in 2m 52s

Quality Gate passed

Issues
8 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 225 in libs/http-datasource/src/datasource-server.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Catch a more specific exception instead of a generic one. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0svRqmm_xGZ5OHuo&open=AZ4H0svRqmm_xGZ5OHuo&pullRequest=164

Check failure on line 136 in libs/pymapget/binding/py-layer.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0szmqmm_xGZ5OHus&open=AZ4H0szmqmm_xGZ5OHus&pullRequest=164

Check warning on line 166 in libs/pymapget/binding/py-layer.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "auto" with an explicit template parameter. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0szmqmm_xGZ5OHut&open=AZ4H0szmqmm_xGZ5OHut&pullRequest=164

Check failure on line 115 in test/integration/python-bindings-smoke.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Features:Map:WayLayer:1:0" 3 times. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0sztqmm_xGZ5OHuu&open=AZ4H0sztqmm_xGZ5OHuu&pullRequest=164

Check warning on line 13 in libs/pymapget/binding/py-datasource.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a name to this namespace. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0szVqmm_xGZ5OHup&open=AZ4H0szVqmm_xGZ5OHup&pullRequest=164

Check warning on line 209 in libs/http-datasource/src/datasource-server.cpp

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This lambda has 23 lines, which is greater than the 20 lines authorized. Split it into several lambdas or functions, or make it a named function. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0svRqmm_xGZ5OHun&open=AZ4H0svRqmm_xGZ5OHun&pullRequest=164

Check warning on line 51 in libs/pymapget/binding/py-layer.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "auto" with an explicit template parameter. 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0szmqmm_xGZ5OHur&open=AZ4H0szmqmm_xGZ5OHur&pullRequest=164

Check warning on line 17 in libs/pymapget/binding/py-datasource.h

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce verbosity with "using enum" for "nlohmann::detail::value_t". 

See more on https://sonarcloud.io/project/issues?id=ndsev_mapget&issues=AZ4H0szVqmm_xGZ5OHuq&open=AZ4H0szVqmm_xGZ5OHuq&pullRequest=164
View more details on SonarQubeCloud