Tuomas Hietanen

Hello, I made a pull request to improve these things a bit. :-)

Hi, just a guess, would `(DateTime)a.Value` work better than `DateTime.Parse(a.Value)` ?

I vote for keeping existing functionality and not adding anything if you really don't need it. :-)

There is already function TryVisitExpressionFunc doing that. Doesn't it work?

I haven't myself used LinqKit in this year at all so I'm not familiar with the latest changes. Back in then, it needed to say `var expanded = IsActive.Expand()` and...

One question is that commands like [allow_facebook_auth](https://www.nuget.org/packages/Microsoft.AspNetCore.Authentication.Facebook) and [use_signalr](https://www.nuget.org/packages/Microsoft.AspNetCore.SignalR/) would add external dependencies, then should these 10 lines-of-code blocks components released as separate nuget packages just for that, or would...

https://github.com/fsprojects/SQLProvider/blob/master/src/SQLProvider.Runtime/Ssdt.DacpacParser.fs ...is the new location. Polyfills was just for old F# compatibility, not needed.

The consuming code is here: https://github.com/fsprojects/SQLProvider/blob/master/src/SQLProvider.Runtime/Providers.MsSqlServer.Ssdt.fs

I could try to replace let scriptRole = role_assignment with something like ```fsharp let scriptRole = let roleName = Guid.NewGuid().ToString() |> ResourceName let keyvaultResourceId = Farmer.ResourceId.create(Farmer.Arm.KeyVault.vaults, (ResourceName keyVaultName)) let roleAssignment...

Meanwhile the docs are still not up-to-date, [this](https://github.com/CompositionalIT/farmer/issues/709#issuecomment-942330540) solved my original issue.