INDAPlus21 / INDAPlus21/azeezd-chess-gui

Pass

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.