-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make C# driver look likes Java driver #4
Conversation
* make use of Expr class in every place a expression is needed * make lambdas instrospection to get parameter names
53b33a7
to
593afc0
Compare
As far I could research, C# only has one single |
You'd still be able to share the connection... Not a big deal now, but more so w/ HTTP 2 or some other more persistent connection protocol. |
yes, also the documentation of |
This is looking good to me. |
/// <param name="port">Port of the FaunaDB server.</param> | ||
/// <param name="timeout">Timeout. Defaults to 1 minute.</param> | ||
/// <param name="secret">Auth token for the FaunaDB server.</param> | ||
/// <param name="clientIO">Optional IInnerClient. Used only for testing.</param>"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type name here is different.
a6c7028
to
2140f40
Compare
I'm waiting for @matt to sign this off. We can ship it, IMO. |
🐑 it |
Don't wrap Set json in {"@set": }
The code is ready to validate, however I don't finished the code documentation yet.
To review the code on Mac you can install Xamarin Studio
brew cask install xamarin-studio
It's possible to run the tests directly on Xamarin, but it may be necessary to install Mono
brew install mono