Suggestion: Avoid render ... and return

[andyw8]

Previously this pattern was used in the Rails docs examples but is now discouraged:

https://github.com/rails/rails/pull/45927/files

[pirj]

It's a controversial topic.
rubocop/ruby-style-guide#730 has been recently merged.
Also related rubocop/ruby-style-guide#785

[mvz]

While .. and .. is now permitted by the default RuboCop settings, the main problem I see with the render ... and return pattern is that it incorrectly suggests we should not return if the render part somehow fails.

[pirj]

Should we recommend return render …?

[andyw8]

I think it's better for the return to be a separate statement. Returning a value would suggest that it's used for something.