Skip to content

NRL-2002 Dynamo Export #1164

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

Draft
jackleary wants to merge 4 commits into
base: develop
Choose a base branch
from

NRL-2002 Update table name

c115f48
Select commit
Loading
Failed to load commit list.
Draft

NRL-2002 Dynamo Export #1164

NRL-2002 Update table name
c115f48
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 5, 2026 in 44s

Quality Gate failed

Failed conditions
2 Security Hotspots
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 226 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this complex logic into separate functions. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQK&open=AZy91PzewoH9RFpg6EQK&pullRequest=1164

Check warning on line 296 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this complex logic into separate functions. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQL&open=AZy91PzewoH9RFpg6EQL&pullRequest=1164

Check warning on line 363 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add logic to this except clause or eliminate it and rethrow the exception automatically. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQM&open=AZy91PzewoH9RFpg6EQM&pullRequest=1164

Check warning on line 90 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQI&open=AZy91PzewoH9RFpg6EQI&pullRequest=1164

Check warning on line 85 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this commented out code. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQH&open=AZy91PzewoH9RFpg6EQH&pullRequest=1164

Check warning on line 192 in terraform/account-wide-infrastructure/modules/dynamo_export/src/glue_job.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this complex logic into separate functions. 

See more on https://sonarcloud.io/project/issues?id=NHSDigital_NRLF&issues=AZy91PzewoH9RFpg6EQJ&open=AZy91PzewoH9RFpg6EQJ&pullRequest=1164
View more details on SonarQubeCloud