Skip to content

Hide http://varnomen.hgvs.org ValueSet errors#97

Merged
ryma2fhir merged 3 commits intomainfrom
ryma2fhir-patch-3
May 22, 2024
Merged

Hide http://varnomen.hgvs.org ValueSet errors#97
ryma2fhir merged 3 commits intomainfrom
ryma2fhir-patch-3

Conversation

@ryma2fhir
Copy link
Copy Markdown
Contributor

@ryma2fhir ryma2fhir commented May 21, 2024

hide any errors relating to the codesystem http://varnomen.hgvs.org within the ValueSet https://terminology.hl7.org/5.5.0/CodeSystem-v3-hgvs.html as the ValueSet is empty but binding is required. This is only an issue with the github validation and not the validator itself.

This issue can be found in https://github.com/NHSDigital/NHSDigital-FHIR-Genomics-ImplementationGuide/actions/runs/9115457743
received: "ERROR None of the codings provided are in the value set 'Human Genome Variation Society (HGVS) Nomenclature' (http://hl7.org/fhir/uv/genomics-reporting/ValueSet/hgvs-vs|3.0.0-ballot), and a coding from this value set is required) (codes = http://varnomen.hgvs.org#NM_001363118.2:c.916G>A) [ Location - Observation.component[3].value.ofType(CodeableConcept)] [ Location - Line[109] Col[6]] ERROR Validation failed for 'http://varnomen.hgvs.org#NM_001363118.2:c.916G>A' [ Location - Observation.component[3].value.ofType(CodeableConcept).coding[0]] [ Location - Line[110] Col[8]]"

Comment thread src/common.js.ts Fixed
@ryma2fhir ryma2fhir changed the title Update common.js.ts Hide http://varnomen.hgvs.org ValueSet errors May 21, 2024
Comment thread src/common.js.ts

// THESE WARNINGS SHOULD ALWAYS BE SILENTLY IGNORED
// Issue with GitHub validator requiring code to be in CodeSystem asset when there is none. No issues with the validator itself. The ValueSet in question is https://terminology.hl7.org/5.5.0/CodeSystem-v3-hgvs.html
if (issue.diagnostics.includes('http://varnomen.hgvs.org')) return false

Check failure

Code scanning / CodeQL

Incomplete URL substring sanitization

'[http://varnomen.hgvs.org](1)' can be anywhere in the URL, and arbitrary hosts may come before or after it.
@ryma2fhir ryma2fhir merged commit da7b14a into main May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants