Skip to content

fix: clean up App.test.tsx conflict markers from Sprint 46 Slice 2 squash#61

Merged
KevinBigham merged 1 commit into
mainfrom
fix/conflict-markers
Apr 30, 2026
Merged

fix: clean up App.test.tsx conflict markers from Sprint 46 Slice 2 squash#61
KevinBigham merged 1 commit into
mainfrom
fix/conflict-markers

Conversation

@KevinBigham
Copy link
Copy Markdown
Owner

Hotfix for typecheck failure introduced when Sprint 46 Slice 2 (PR #51) was rebased+squash-merged with leftover conflict markers in apps/web/src/app/App.test.tsx (lines 117/124/129). Keeps both Sprint 46 Slice 1 emitter-mount test and Slice 2 achievements route test, and updates the route assertion to match the actual additive routeTree.addChildren pattern.

…nt 46 Slice 2 squash merge

The Sprint 46 Slice 2 (achievements gallery) squash-merge missed conflict markers in App.test.tsx.
This hotfix keeps both new test cases (Sprint 46 Slice 1's emitter-mount test and Slice 2's
achievements route test), and updates the route assertion to match the actual additive
routeTree.addChildren pattern that resolved against Sprint 45's prior addChildren calls.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@KevinBigham KevinBigham merged commit dacda0a into main Apr 30, 2026
2 checks passed
@KevinBigham KevinBigham deleted the fix/conflict-markers branch April 30, 2026 19:08
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