Skip to content

test(review): default pull request fixtures - #446

Merged
caspar-magi merged 1 commit into
mainfrom
refactor/default-review-fixtures
Jul 28, 2026
Merged

test(review): default pull request fixtures#446
caspar-magi merged 1 commit into
mainfrom
refactor/default-review-fixtures

Conversation

@hirotomoyamada

Copy link
Copy Markdown
Member

AI used

  • I checked the generated content before submitting.

Description

Simplify shared review fixtures by providing default pull request checks and metadata.

Current behavior (updates)

Review and merge tests repeatedly constructed empty checks and pull request metadata.

New behavior

Each review fixture starts with independent default checks and metadata; tests override only the fields relevant to their scenario.

Is this a breaking change (Yes/No):

No.

Additional Information

Verified with targeted review and merge suites, type checking, and formatting.

@hirotomoyamada
hirotomoyamada requested review from a team July 28, 2026 23:43
@github-actions github-actions Bot added the magi:merge The command "magi:merge" is currently running label Jul 28, 2026
@caspar-magi
caspar-magi added this pull request to the merge queue Jul 28, 2026
Merged via the queue into main with commit 9d12004 Jul 28, 2026
11 checks passed
@caspar-magi
caspar-magi deleted the refactor/default-review-fixtures branch July 28, 2026 23:47
@github-actions github-actions Bot removed the magi:merge The command "magi:merge" is currently running label Jul 28, 2026
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.

4 participants