Replies: 1 comment
-
I think its reasonable to go that way, but I really don't want to do another major release soon. What we could do without breaking change is to introduce another derive that combines |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hey!
Recently I've been migrating Stockly's code base to use
#[derive(Selectable)]
. I'm not aware of use cases where a structure can implementSelectable
without implementingQueryable
. Therefore I think theSelectable
should by default add aQueryable
implementation with an optout as#[diesel(no_impl_queryable)]
.Pros:
Cons:
Beta Was this translation helpful? Give feedback.
All reactions