Merge pull request rails#48448 from a5-stable/fix-returning-doc

wrap Arel.sql to avoid error in doc of insert_all, upsert_all

Author: guilleiguaran

activerecord/lib/active_record/persistence.rb

@@ -122,7 +122,7 @@ def insert(attributes, returning: nil, unique_by: nil, record_timestamps: nil)
       #   clause entirely.
       #
       #   You can also pass an SQL string if you need more control on the return values
-      #   (for example, <tt>returning: "id, name as new_name"</tt>).
+      #   (for example, <tt>returning: Arel.sql("id, name as new_name")</tt>).
       #
       # [:unique_by]
       #   (PostgreSQL and SQLite only) By default rows are considered to be unique
@@ -212,7 +212,7 @@ def insert!(attributes, returning: nil, record_timestamps: nil)
       #   clause entirely.
       #
       #   You can also pass an SQL string if you need more control on the return values
-      #   (for example, <tt>returning: "id, name as new_name"</tt>).
+      #   (for example, <tt>returning: Arel.sql("id, name as new_name")</tt>).
       #
       # [:record_timestamps]
       #   By default, automatic setting of timestamp columns is controlled by
@@ -278,7 +278,7 @@ def upsert(attributes, on_duplicate: :update, returning: nil, unique_by: nil, re
       #   clause entirely.
       #
       #   You can also pass an SQL string if you need more control on the return values
-      #   (for example, <tt>returning: "id, name as new_name"</tt>).
+      #   (for example, <tt>returning: Arel.sql("id, name as new_name")</tt>).
       #
       # [:unique_by]
       #   (PostgreSQL and SQLite only) By default rows are considered to be unique