Skip to content

factory(32): #33 Pagination helper - #129

Merged
shadetheory merged 1 commit into
masterfrom
factory/32
Oct 26, 2026
Merged

shadetheory merged 1 commit into
masterfrom
factory/32

Conversation

@shadetheory

Copy link
Copy Markdown
Owner

Factory job 32 (pfc), model ollama/qwen3-coder-next:48k.

Checks:

  • vendor/bin/pint --test: pass
  • vendor/bin/phpstan analyse --memory-limit=1G: pass
  • vendor/bin/pest: pass

Diff lines: 288

opened by local-factory; do not auto-merge

@shadetheory shadetheory added the risk:low Routine change label Oct 25, 2026

@lexi-chief-of-staff lexi-chief-of-staff Bot 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.

Ship it.

  • Matches the established pagination pattern from IssuesResource::allForRepo (identical guard, header parsing, loop).
  • Tests exercise multi-page, single-page, empty, param passthrough, and max-page guard.
  • No new public surface or signature changes.

@shadetheory

Copy link
Copy Markdown
Owner Author

Quality Gate: success

All quality checks passed.

Quality Check Results

All tools passed without violations.

@shadetheory
shadetheory merged commit 041da9e into master Oct 26, 2026
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:low Routine change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant