Skip to content

Removed IDeepCopyable - #2996

Merged
ewoutkramer merged 17 commits into
develop-6.0from
feature/remove-deepcopyable
Jan 16, 2025
Merged

Removed IDeepCopyable#2996
ewoutkramer merged 17 commits into
develop-6.0from
feature/remove-deepcopyable

Conversation

@Kasdejong

@Kasdejong Kasdejong commented Dec 17, 2024

Copy link
Copy Markdown
Member

Description

Removed IDeepCopyable in favor of Base (Base still implements the methods previously defined on IDeepCopyable by means of extension methods)

Related issues

Closes #2938

⚠️ Review FirelyTeam/fhir-codegen#51 as well

@Kasdejong
Kasdejong changed the base branch from develop to develop-6.0 December 17, 2024 15:00
Comment thread src/Hl7.Fhir.Base/Model/DynamicDataType.cs Outdated
Comment thread src/Hl7.Fhir.Conformance/Specification/Snapshot/ElementDefnMerger.cs Outdated
Comment thread src/Hl7.Fhir.STU3.Tests/Model/DeepCopyTest.cs Outdated
Comment thread src/Hl7.Fhir.STU3/Specification/Snapshot/ElementDefnMerger.cs Outdated
Comment thread src/Hl7.Fhir.Shared.Tests/Model/DeepCopyTest.cs Outdated
Comment thread src/Hl7.Fhir.Support.Poco.Tests/Model/ModelTests.cs
@Kasdejong
Kasdejong requested a review from ewoutkramer January 14, 2025 09:44
Kasdejong and others added 6 commits January 14, 2025 13:39
# Conflicts:
#	src/Hl7.Fhir.Base/CompatibilitySuppressions.xml
# Conflicts:
#	src/Hl7.Fhir.Base/CompatibilitySuppressions.xml
#	src/Hl7.Fhir.Base/Model/CodeOfT.cs
@ewoutkramer
ewoutkramer enabled auto-merge January 16, 2025 10:42
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.

Remove IDeepCopyable interface

3 participants