Skip to content

doc: add README - #3

Merged
kbond merged 1 commit into
zenstruck:1.xfrom
kbond:docs
Aug 20, 2026
Merged

kbond merged 1 commit into
zenstruck:1.xfrom
kbond:docs

Conversation

@kbond

@kbond kbond commented Oct 26, 2021 •

Copy link
Copy Markdown
Member

The package has no documentation, so this adds a full README: the Collection API and its implementations, pagination, the Doctrine integrations (EntityResult, repositories, batch processing, the doctrine/collections bridge and specifications), the Symfony bundle and static analysis.

Grid is intentionally left out - it's the least settled part of the API and deserves its own pass.

Also refreshes the composer.json description/keywords and fixes the suggest entries: two referenced classes that don't exist (CollectionDecorator, CollectionAdapter), and symfony/expression-language, zenstruck/uri and doctrine/doctrine-bundle were missing despite hard-failing when absent.

@kbond kbond added the documentation Improvements or additions to documentation label Oct 26, 2021
@tacman

tacman commented Jun 10, 2024

Copy link
Copy Markdown

yes, even if it's just a "install and here will be your collection's new super-powers". Thx.

@kbond kbond changed the title Add readme doc: add README Aug 16, 2026
@kbond
kbond marked this pull request as ready for review August 16, 2026 22:01
Also refreshes the `composer.json` description/keywords and fixes `suggest` entries that referenced classes which don't exist.

@bocharsky-bw bocharsky-bw left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This package definitely needed docs, and this PR adds a decent amount of docs! That's a great start 👍🏼

@kbond
kbond merged commit a80a900 into zenstruck:1.x Aug 20, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.

3 participants