jeremyevans/sequel — pull request #390 by jfirebaugh (2011-10-28 00:39:44+00:00)
Title: Change #each to always yield model instances

Opening a pull request to gauge your feeling on this change; I think you've said you were open to the idea when it was previously discussed, and it looks pretty easy to do.

I'm upgrading an app from 3.22 which is affected by the backward compatibility changes that came with eager graphing performance improvements (99beb68ecef93dd6d721ce74cc3463f20a516c72). Having each yield model instances in all cases would help ease the upgrade for me.

I tested core, model, integration, mssql, and plugin specs. All are passing except for 2 failures in the LazyAttributes plugin which I'm not sure of the cause.