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

Already on GitHub? Sign in to your account

Delete should respect 'limit' #104

wants to merge 1 commit into


None yet
2 participants

I was surprised to find limit(...).delete_all didn't work so I added the following limit recognition to the rails arel libs.

I'm sure there was a reason for omitting 'limit' from the delete statements, but just in case it should be there, here's a patch that adds the functionality.


rafaelfranca commented Feb 24, 2015

Closed by #356


rafaelfranca commented Feb 24, 2015

Thank you so much for the pull request.

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