Move Sync-over-Async into Scripting

fsprojects / FSharp.AWS.DynamoDB

F# wrapper API for AWS DynamoDB

MIT License

58 stars 18 forks source link

Closed bartelink closed 2 years ago

bartelink commented 2 years ago

Moved Sync-over-Async delegating wrappers of the fundamentally Async operations in TableContext into namespace FSharp.AWS.DynamoDB.Scripting

bartelink commented 2 years ago

For me, this ticks two boxes:

removes noise from the impl so I can traverse it more easily
means one correctly considers the asynchronous nature intrinsic to these operations in your real app logic (I believe the F# Async guide, along with other such documents advise against the inclusion of such helpers)

I'm not emotionally attached to this; if people are not interested in the breaking change, I'm happy to abandon it

bartelink commented 2 years ago

I suspect it’s only scripting scenarios that would have ever used them

There is the small matter of the tests, which major on using the Sync versions, but I'm reluctant to go rewriting those (for now anyway!)

Thanks!