Good point! Fixed and submitted PR, thanks for reviewing so quickly

On Tue, Nov 20, 2018 at 5:00 PM Jeremy Evans <[email protected]> wrote:

> On Tuesday, November 20, 2018 at 4:30:23 PM UTC-8, Ben Alavi wrote:
>>
>> Patch that would take a new :qualify_tables option to the CTI plugin:
>>
>> https://gist.github.com/benalavi/159110fd58907c7925fc8bd1958e3ad2
>>
>> So the API would be:
>>
>> plugin :class_table_inheritance, qualify_tables: true
>>
>
> This looks good.  Note that you do not need to freeze the newly added
> attribute, true/false is always considered frozen.
>
> Thanks,
> Jeremy
>
> --
> You received this message because you are subscribed to a topic in the
> Google Groups "sequel-talk" group.
> To unsubscribe from this topic, visit
> https://groups.google.com/d/topic/sequel-talk/Tikx3GRUVMU/unsubscribe.
> To unsubscribe from this group and all its topics, send an email to
> [email protected].
> To post to this group, send email to [email protected].
> Visit this group at https://groups.google.com/group/sequel-talk.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sequel-talk.
For more options, visit https://groups.google.com/d/optout.

Reply via email to