elastic / elasticsearch-rs

Official Elasticsearch Rust Client
https://www.elastic.co/guide/en/elasticsearch/client/rust-api/current/index.html
Apache License 2.0
705 stars 72 forks source link

Builder associated fns accept T instead of Option<T> #23

Closed russcam closed 4 years ago

russcam commented 4 years ago

This commit updates api_generator to generate all associated functions for builder structs to be T rather than Option<T>. This makes the API more ergonomic to use at the expense that a builder cannot be cloned and values set on the clone to Option::<T>::None.

Closes #17