Skip to content

Allow custom win target#4

Open
phuetz wants to merge 1 commit into
mainfrom
codex/ajouter-un-paramètre-target-à-checkwin

Hidden character warning

The head ref may contain hidden characters: "codex/ajouter-un-param\u00e8tre-target-\u00e0-checkwin"
Open

Allow custom win target#4
phuetz wants to merge 1 commit into
mainfrom
codex/ajouter-un-paramètre-target-à-checkwin

Conversation

@phuetz
Copy link
Copy Markdown
Owner

@phuetz phuetz commented Jun 14, 2025

Summary

  • expose target param in checkWin
  • track target value in GameState
  • compute target in initGame based on board size
  • use gameState.targetValue when checking for a win

Testing

  • npm run lint (fails: expo not found)
  • npx tsc -p tsconfig.json --noEmit (fails: missing modules and JSX config)

https://chatgpt.com/codex/tasks/task_e_682cff05bd488323a90b58b999b1550a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant