Change Hl7.Fhir.Model.Task into FhirTask so it doesn't conflict with System.Threading.Tasks.Task #3091
almostchristian
started this conversation in
Ideas
Replies: 1 comment 1 reply
-
Yes, it's a nuisance in our code too. We are hardly ever using Task, yet including the Hl7.Fhir.Model namespace makes async C# give compile errors if yoiu don't carefully use aliases. It's of course a major breaking change, though one could wonder how many people of the SDK actually use Task (there will be some for sure) - but even they might agree ;-0 I'll discuss it with the team - and we should discuss this in the community as well. We're putting the finishing touches on SDK 6.0, so if we do it, it should be in that major release, otherwise we will have to wait for SDK 7.0, easily 18-24 months out. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Because of the naming, code in that has a
using Hl7.Fhir.Model;
needs an accompanyingusing Task = System.Threading.Tasks.Task
. This is tripping up some of our users that are trying out LLM generated code using the Firely SDK.Beta Was this translation helpful? Give feedback.
All reactions