Skip to content
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.

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

TODO fix explicitly loading exceptations, autoload removed #1845

Closed
wants to merge 11 commits into from

Conversation

vishnuatrai
Copy link
Contributor

No description provided.

@pixeltrix
Copy link
Contributor

And where are the explicit requires in the other files?

@vishnuatrai
Copy link
Contributor Author

where else you think its auto loading?

@josevalim
Copy link
Contributor

@vatrai the TODO says that each file that raises one of those exceptions, should require the exception file manually.

@vishnuatrai
Copy link
Contributor Author

thanks! @josevalim, got it. I will add more commits for that in this pull request.

jake3030 pushed a commit to jake3030/rails that referenced this pull request Jun 28, 2011
…solved]

Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
@vishnuatrai
Copy link
Contributor Author

oops, something happened wrong. closing this pull request, will create new request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants