Skip to content

Document native collections without changing present semantics - #45

Merged
dblock merged 1 commit into
ruby-grape:masterfrom
dblock:docs/native-collections-18
Oct 27, 2026
Merged

dblock merged 1 commit into
ruby-grape:masterfrom
dblock:docs/native-collections-18

Conversation

@dblock

@dblock dblock commented Oct 27, 2026

Copy link
Copy Markdown
Member

Supersedes #18.

Use the current Roar collection factory explicitly: Representer.for_collection.prepare(records).to_hash(user_options: { env: env }). Verify modules and decorators, populated and empty arrays, request-aware item links, and unchanged HAL collection envelopes through actual requests.

The automatic Array dispatch proposed by #18 changes existing collection representers into item representers and breaks collection properties/envelopes. Preserve that existing behavior instead of making a silent compatibility change. No Travis changes, removed request context, or new serialization API are necessary.

AI-authored on behalf of @dblock.

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@dblock
dblock force-pushed the docs/native-collections-18 branch from 590088e to 3e73533 Compare October 27, 2026 10:19
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the docs/native-collections-18 branch from 3e73533 to cf602f1 Compare October 27, 2026 10:28
@dblock
dblock merged commit bc73375 into ruby-grape:master Oct 27, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant