render_collection

[leastbad]

Type of PR (feature, enhancement, bug fix, etc.)

Enhancement

Description

Renames the Reflex#wrap method to render_collection.

Render collection method signature now includes a single resource passed, so:

wrap(render(@posts), @posts) now becomes render_collection(@posts)

I created a temporary wrap method that issues a stern deprecation warning, even though technically wrap has never been part of a shipped release. It issues a warning that is hard to ignore and then passes to render_collection. We'll remove this before shipping the final v3.5.

Why should this be added

I strongly suspect that 90% of wrap calls will actually receive the same parameter essentially twice. You can still call it the old way, but if you cut through to general intent, this is clearly a superior name and mechanism.

Checklist

• My code follows the style guidelines of this project
• Checks (StandardRB & Prettier-Standard) are passing
• This is not a documentation update

[hopsoft]

Looks great. I think we should remove wrap and the deprecation warning as its never been released.