AR Relation docs

Hi guys,

I’m covering this file, and I’ve noticed that the bang version of the methods are not prefixed by no-doc.

Example:

includes!

preload!

join!

etc.

Should I go ahead and add no-docs to those?

Depends. How are they different to their non-bang variants?

in general, the non bang version allows for empty arguments, and calls the bang version if everything looks good.

Example:

https://github.com/rails/rails/blob/b6ce2306cdbc5b074b75d5a22d6bfdc83a6a037a/activerecord/lib/active_record/relation/query_methods.rb#L181-190

In that case then, yeah I would nodoc the bang versions

will do