Add better errors reporting for ActiveModel::Configuration #5736

Merged
merged 1 commit into from Apr 11, 2012

Conversation

Projects
None yet
3 participants
Contributor

oscardelben commented Apr 4, 2012

I was unsure about adding it to the other two instances of class_eval as well, thoughts?

Owner

rafaelfranca commented Apr 4, 2012

Could you add tests for it?

Contributor

oscardelben commented Apr 4, 2012

@rafaelfranca I don't think we test this kind of stuff.

Owner

rafaelfranca commented Apr 4, 2012

Ok. I'll ask to someone to merge it.

jonleighton merged commit e177525 into rails:master Apr 11, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment