Feature/ndit 602 xsd validation #15

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

georgeRobertson merged 6 commits into develop_v04 from feature/ndit-602_xsd_validation

Dec 3, 2025

Merged

Feature/ndit 602 xsd validation #15

fix: fix spark xsd check to raise messageBearingError

SonarQubeCloud / SonarCloud Code Analysis succeeded Dec 3, 2025 in 33s

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 92 in src/dve/core_engine/backends/implementations/spark/readers/xml.py

sonarqubecloud / SonarCloud Code Analysis

Specify both "master" and "appName" parameters to initialize this SparkSession.

See more on https://sonarcloud.io/project/issues?id=NHSDigital_data-validation-engine&issues=AZrjWbBVu-4h0EyNNpHP&open=AZrjWbBVu-4h0EyNNpHP&pullRequest=15

Check warning on line 77 in src/dve/core_engine/backends/implementations/spark/readers/xml.py

sonarqubecloud / SonarCloud Code Analysis

Method "__init__" has 17 parameters, which is greater than the 13 authorized.

See more on https://sonarcloud.io/project/issues?id=NHSDigital_data-validation-engine&issues=AZrA-ejpT-V0F1DOJ4tF&open=AZrA-ejpT-V0F1DOJ4tF&pullRequest=15

Check warning on line 47 in src/dve/core_engine/backends/readers/xml_linting.py

sonarqubecloud / SonarCloud Code Analysis

Replace this map call with a comprehension.

See more on https://sonarcloud.io/project/issues?id=NHSDigital_data-validation-engine&issues=AZrA-emFT-V0F1DOJ4tG&open=AZrA-emFT-V0F1DOJ4tG&pullRequest=15

View more details on SonarQubeCloud