Yeah :) I see the core build green now. But I meant if we have any particular reason for setting that `:null => false`? I believe it makes more sense to not add that constraint into db level unless we have a pretty good reason to do it.

Except for that particular option eveything looks great to me here. Thanks for the patch.