Feedback

The `.all` here is unnecessary. In older versions of Active Record this would actually cause an error because `.all` would return an array rather than an Active Record query relation object, meaning that you wouldn't be able to chain the rest of those methods (`.limit`, `.offset`, etc) on the end of it.

So while it won't cause an error in new versions of Active Record, this is only a very recent change and it's very possible that you'll end up working somewhere on an app that isn't on the latest version of Active Record, where this wouldn't work.