Skip to content

Conversation

@ewoutkramer
Copy link
Contributor

@ewoutkramer ewoutkramer commented Nov 14, 2025

Firely has released SDK6, so it's time to donate the changes from Firely's fhir-codegen back into MS codegen!

⚠️ Contains a few changes to the non-Firely parts. Although minor, you might want to take a look at them. (and yes, some of them we discussed together a year and a half ago, in some beautiful green US state 🤣 )

mmsmits and others added 30 commits October 2, 2024 11:24
Remove unused files (they have been moved)
Removed generation of nested/resource args to fhirtype.
…children

(For 2973) Remove Children/Namedchildren
# Conflicts:
#	src/Microsoft.Health.Fhir.CodeGen/Language/Firely/CSharpFirely2.cs
…ct-shared-properties

# Conflicts:
#	src/Microsoft.Health.Fhir.CodeGen/Language/Firely/CSharpFirely2.cs
…perties

Generalize handling of Base/Conformance properties that changed type
Also made dictionary-like operations public.
…parer

# Conflicts:
#	src/Microsoft.Health.Fhir.CodeGen/Language/Firely/CSharpFirely2.cs
ewoutkramer and others added 28 commits April 23, 2025 16:42
No longer touches the overflow when copying pocos when its empty.
Small correction in name of one of the launchsettings
Changes to be able to generate FHIR R6.
# Conflicts:
#	src/fhir-codegen/Properties/launchSettings.json
Changes to codegen to make R6 satellites compile
# Conflicts:
#	src/Microsoft.Health.Fhir.CodeGen/Language/Firely/CSharpFirely2.cs
#	src/fhir-codegen/Properties/launchSettings.json
…s-not-nullable

made required elements not nullable in the model
Create special allowed types attributes for open choice types
Skip generation of abstract enum members
@ewoutkramer ewoutkramer changed the title Update Microsoft fhir-codegen with SDK6 codegen Update Microsoft fhir-codegen with Firely SDK6 codegen Nov 14, 2025
@ewoutkramer
Copy link
Contributor Author

@GinoCanessa - would you mind reviewing and pulling this PR?

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.

4 participants