Skip to content
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.

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

CH-Core dependsOn CH-EPR-TERM current #65

Closed
oliveregger opened this issue Aug 10, 2020 · 4 comments
Closed

CH-Core dependsOn CH-EPR-TERM current #65

oliveregger opened this issue Aug 10, 2020 · 4 comments

Comments

@oliveregger
Copy link
Contributor

Aktuell hängt CH-Core vom Implementierungsleitfaden CH-EPR-TERM als current ab, das heisst es ist immer die letzte Version vom CI die verwendet wird.

Dies führte zum Fehler von #63, dass der 1001 Code ungültig ist.

1.) Kann beim Validieren eine andere Version von CH-EPR-Term angegeben werden?
2.) Sollte auf eine fixe Version von CH-EPR-TERM abhängig gemacht werden?
2b) Kann allenfalls eine minimale Version angegeben werden die neuere auch erlaubt?

Aufnehmen in Zulip wie es andere machen.

Nebenbemerkung: In der Art-Decor ValueSet Definition können einzelne Codes nicht als inaktiv deklariert werden.

@ziegm ziegm added this to the STU2 milestone Aug 10, 2020
@ziegm ziegm added the STU 1.2.0 Ballot Comments/Issues from the Ballot for STU2 label Aug 10, 2020
@oliveregger
Copy link
Contributor Author

Gibt aktuell noch keine Lösung, ausser dass current in Zukunft nicht erlaubt ist.

Diskussion hier gestartet: https://chat.fhir.org/#narrow/stream/179239-tooling/topic/NPM.20proposed.20changes

@oliveregger
Copy link
Contributor Author

In the FHIR package spec, we did specify wildcards for patches to solve floating dependencies:
https://confluence.hl7.org/pages/viewpage.action?pageId=35718629#NPMPackageSpecification-Versionreferences

this functionality as been added via a PR to the Validator/Publisher and now a version of 2.0.x can be specified as a dependency.

@oliveregger
Copy link
Contributor Author

@oliveregger
Copy link
Contributor Author

fix ch-core dependency to ch-epr-term version 2.0.x instead of current. agree - 11 - abstain - 0 -negative -0

@ziegm ziegm removed the STU 1.2.0 Ballot Comments/Issues from the Ballot for STU2 label Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants