Skip to content

Commit 83bc142

Browse files
committed
review changes
1 parent 65617e0 commit 83bc142

File tree

5 files changed

+6
-39
lines changed

5 files changed

+6
-39
lines changed

lambdas/delta_backend/pyproject.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@ description = ""
55
authors = ["Your Name <[email protected]>"]
66
readme = "README.md"
77
packages = [
8-
{include = "src"}
8+
{include = "src"},
9+
{include = "common", from = "../shared/src"}
910
]
1011

1112
[tool.poetry.dependencies]

lambdas/delta_backend/src/extractor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ class Extractor:
1414
)
1515
EXTENSION_URL_SCT_DESC_DISPLAY = "https://fhir.hl7.org.uk/StructureDefinition/Extension-UKCore-CodingSCTDescDisplay"
1616

17-
CODING_SYSTEM_URL_SNOMED = "http://snomed.info/sct" # NOSONAR(S5332) this is a code system identifier, not a URL
17+
CODING_SYSTEM_URL_SNOMED = "http://snomed.info/sct"
1818
ODS_ORG_CODE_SYSTEM_URL = "https://fhir.nhs.uk/Id/ods-organization-code"
1919
DEFAULT_LOCATION = "X99999"
2020
NHS_NUMBER_SYSTEM_URL = "https://fhir.nhs.uk/Id/nhs-number"

lambdas/shared/src/common/utils.py

Lines changed: 0 additions & 20 deletions
This file was deleted.

lambdas/shared/tests/test_common/test_utils.py

Lines changed: 0 additions & 13 deletions
This file was deleted.

sonar-project.properties

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,10 @@ sonar.python.version=3.11
66
sonar.exclusions=**/e2e/**,**/e2e_batch/**,**/devtools/**,**/proxies/**,**/utilities/scripts/**,**/infrastructure/account/**,**/infrastructure/instance/**,**/infrastructure/grafana/**,**/terraform_aws_backup/**,**/tests/**
77
sonar.python.coverage.reportPaths=backend-coverage.xml,delta-coverage.xml,ack-lambda-coverage.xml,filenameprocessor-coverage.xml,recordforwarder-coverage.xml,recordprocessor-coverage.xml,mesh_processor-coverage.xml,redis_sync-coverage.xml,mns_subscription-coverage.xml,id_sync-coverage.xml,shared-coverage.xml,batchprocessorfilter-coverage.xml
88
sonar.cpd.exclusions=**/cache.py,**/authentication.py,**/test_cache.py,**/test_authentication.py,**/mns_service.py,**/errors.py,**/Dockerfile,lambdas/shared/src/common/**,filenameprocessor/src/logging_decorator.py,backend/src/fhir_service.py
9-
sonar.issue.ignore.multicriteria=exclude_snomed_urls,exclude_hl7_urls
9+
sonar.issue.ignore.multicriteria=exclude_snomed_urls,exclude_hl7_urls,exclude_safe_tmpdirs
1010
sonar.issue.ignore.multicriteria.exclude_snomed_urls.ruleKey=python:S5332
1111
sonar.issue.ignore.multicriteria.exclude_snomed_urls.resourceKey=**http://snomed\.info/sct**
1212
sonar.issue.ignore.multicriteria.exclude_hl7_urls.ruleKey=python:S5332
1313
sonar.issue.ignore.multicriteria.exclude_hl7_urls.resourceKey=**http://terminology\.hl7\.org/CodeSystem/v3-NullFlavor**
14-
sonar.issue.ignore.multicriteria=e1
15-
sonar.issue.ignore.multicriteria.e1.ruleKey=python:S5443
16-
sonar.issue.ignore.multicriteria.e1.resourceKey=**/mns_setup.py
14+
sonar.issue.ignore.multicriteria.exclude_safe_tmpdirs.ruleKey=python:S5443
15+
sonar.issue.ignore.multicriteria.exclude_safe_tmpdirs.resourceKey=**/mns_setup.py

0 commit comments

Comments
 (0)