On Friday, April 24, 2015 at 7:33:58 PM UTC-7, Jeremy Evans wrote: > > On Friday, April 24, 2015 at 4:24:31 PM UTC-7, Quinn Harris wrote: >> >> Jeremy, >> >> Consider this plugin as a replacement for the class_table_inheritance >> plugin. I intended it to be a drop in replacement and it passes all the >> specs for class_table_inheritance except the one not allowing anonymous >> subclasses. It also uses single_table_inheritance so it has less >> duplicated functionality than the current class_table_inheritance. The >> commit history should be helpful to see the changes made from the >> class_table_inheritance code. I can prepare a pull request if you would >> like. >> > > Quinn, > > I'm definitely open to replacing class_table_inheritance code with this > code if it passes all tests. I'd just like to get some community feedback > before doing that. > > If any class_table_inheritance users could try this out and let me know if > it works for them, and their thoughts about this in general, I would > appreciate it. >
Quinn, Since nobody appears to have objections to this, can you send a pull request that makes the class table inheritance plugin also support single table inheritance? Thanks, Jeremy -- 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 http://groups.google.com/group/sequel-talk. For more options, visit https://groups.google.com/d/optout.
