machinist icon indicating copy to clipboard operation
machinist copied to clipboard

Published 20 hours ago • verified publisher icon notahat

Sequel adapter

Open jfirebaugh opened this issue 13 years ago • 4 comments

I added a Sequel adapter.

Since Sequel doesn't support deferred associations natively, the adapter is slightly more complex than the ActiveRecord adapter.

jfirebaugh avatar Jul 13 '11 21:07 jfirebaugh

Is it possible to get this merged in? It would be really nice to have Sequel support in machinist...

matthauck avatar Oct 31 '12 05:10 matthauck

This would be better packaged as a separate gem, perhaps machinist-sequel—it's cleanly independent of machinist core and this way, the mainenance of one won't affect the other.

benhoskings avatar Jan 26 '13 04:01 benhoskings

Seeing as ActiveRecord is supported as an adapter already, and not really part of the "core", is it not reasonable to have Sequel also supported in the main gem?

matthauck avatar Jan 28 '13 07:01 matthauck

Well, activerecord should be moved to a separate gem too, but that's a separate job :)

Since the two concerns can be cleanly separated I think separate gems would be better.

On 28/01/2013, at 6:07 PM, Matt Hauck [email protected] wrote:

Seeing as ActiveRecord is supported as an adapter already, and not really part of the "core", is it not reasonable to have Sequel also supported in the main gem?


Reply to this email directly or view it on GitHub.

benhoskings avatar Jan 28 '13 09:01 benhoskings