search

edgedb / edgedb-rust

The official Rust binding for EdgeDB
https://edgedb.com
Apache License 2.0
209 stars 26 forks source link

encode missing entry as `None` #322

Closed MrFoxPro closed 2 months ago

MrFoxPro commented 2 months ago

Otherwise we need to specify every None entry explicitly. I think this is not very convinent and also it doesn't align with javascript client behaviour

MrFoxPro commented 2 months ago

I don't like this because it is implicit. When you don't specify an argument, it is very likly that it was forgotten and bug. I understand that is it easy to omit an argument, but I'm not convinced about that this change is beneficial in the long-run.

do you think it could be configured via client config?

aljazerzen commented 2 months ago

do you think it could be configured via client config?

This would be hard to implement and also quite messy.