-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feat/maintenance #159
Feat/maintenance #159
Conversation
@@ -6,7 +6,7 @@ | |||
"experimental": false, | |||
"version": "3.0.0", | |||
"publisher": "gematik GmbH", | |||
"date": "2023-07-01", | |||
"date": "2023-11-28", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Datum würde ich nur bei Inkrementierung ändern. Dann aber auch als TC mit Releasenotes etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
date
gibt an, wann das Profil zuletzt ein Update erfahren hat. Da wir in diesem Fall Links fixen, würde ich es setzen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s.o-
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sollten wir hier nicht eine ausführliche Description der Operation mitführen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Würde an dieser Stelle die Description im IG verwenden. Dann müssen wir diese nicht doppelt pflegen 👍
Resources/input/fsh/rulesets.fsh
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s.o.
Resources/sushi-config.yaml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im Basismodul führen wir noch:
applyExtensionMetadataToRoot: false
Kannst DU das einordnen? Fehlt es hier?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sollte auf Default belassen werden, fehlt daher nicht :-)
239adc5
to
2918a07
Compare
No description provided.