New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Require active_record #139

Merged

parndt merged 1 commit into collectiveidea:master from bogdan:master

May 15, 2012

Contributor

bogdan commented May 15, 2012

My bundle console command failing due to this missing require statement.


          Require AR if not yet loaded.

989fb1a

Collaborator

parndt commented May 15, 2012

You know, I don't think we should be extending all of ActiveRecord::Base anymore. However, we'll address that in a future version like 2.1 or 3

parndt added a commit that referenced this pull request


          Merge pull request #139 from bogdan/master

8a9619a

Require active_record

parndt merged commit 8a9619a into collectiveidea:master

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

Labels

None yet