WIP: Add UUID array support

Zaid-Ajaj / Npgsql.FSharp.Analyzer

F# analyzer that provides embedded SQL syntax analysis, type-checking for parameters and result sets and nullable column detection when writing queries using Npgsql.FSharp.

MIT License

137 stars 9 forks source link

WIP: Add UUID array support #1

Closed baronfel closed 3 years ago

baronfel commented 4 years ago

Proposed Changes

Add support for array types (and uuid array) to the suggestions in the analyzer.

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist

[ ] Build and tests pass locally
[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Further comments

Currently fails on the new test I added.

Zaid-Ajaj commented 3 years ago

Hi @baronfel I went ahead and implement array support for uuid, text and int along with other goodies like better error messages and handling empty result sets. It is available on nuget as of :rocket: v3.11.0 :rocket: give it a try and let me how it works for you and of course whether I missed something :smile:

Check out the releases page in case you want the VS extension

I will close this PR since the functionality is now implemented