search

cornucopia-rs / cornucopia

Generate type-checked Rust from your PostgreSQL.
Other
759 stars 31 forks source link

Rename `params` to `param_struct` #155

Closed LouisGariepy closed 1 year ago

LouisGariepy commented 1 year ago

I think it makes the concept more explicit. We could also rename the trait Params to ParamStruct

LouisGariepy commented 1 year ago

@Virgiel Looking at https://github.com/cornucopia-rs/cornucopia/issues/154#issuecomment-1251623171 it seems you favor renaming bind to args and params to struct, is that right?

Virgiel commented 1 year ago

I develop the name change proposal further in #154.

LouisGariepy commented 1 year ago

Closing in favor of #154. This present issue's scope is narrower, so I doesn't really make sense to discuss it separately from the broader changes proposed in #154.