INDAPlus21 / INDAPlus21/azeezd-chess-gui
Pass
- Dominant language
- Rust
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
**Very well done Azeez!**
The graphics are 👌. Once again, nice moduling and good in-code documentation.
Curious, with all the times you set `pending_promotion_move` to `PendingMove { _from: "".to_string(), _to: "".to_string() }`, why not just declare the field as `Option`?
Also, a quicker way of initialising an empty `String`, is not to convert an empty `&str`, but to call the constructor `String::new`.
And, what if I would like to restart the game in mid-game, as of the result of a tie.
Contributor guide
No contributing guide indexed for this repository
Research direction
The issue mentions the pending_promotion_move field and asks about restarting the game during play, but it names no files, tests, or entry points. First locate the game-state handling for pending promotions and restart behavior; the scope and completion criteria need clarification before implementation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- game-dev
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100