On May 28, 2008, at 12:36 PM, Ryan Davis wrote: > > On May 28, 2008, at 09:07 , Dave Thomas wrote: > >> 2. I'm being more opinionated when it comes to the ways you should >> write code. So, for example, I'm pushing class inheritance way >> back, and instead focusing on mixins as the preferred way of >> structuring the sharing of functionality. (Obviously, inheritance >> still has a place, but I'd like to see less instances of code like >> "class Product < ActiveRecord::Base') > > How about you just push against nonsensical names like Base? > > class Product < ActiveRecord > > makes _plenty_ of sense by itself. :P class Product include ActiveRecord end makes even more... After all, a Product is not an ActiveRecord--it uses the functionality. Dave