Skip to content

Conversation

@CamCranda11
Copy link

Added the necessary code, a few printlns, into the SudokuGame.java file in lines 22-24 to address issue #23

JoshLapierre45 and others added 17 commits October 1, 2025 14:48
name

Signed-off-by: hchagerman <123512775+hchagerman@users.noreply.github.com>
add if Score == null

Signed-off-by: hchagerman <123512775+hchagerman@users.noreply.github.com>
Update GameHistoryTracker.java
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Copy link
Contributor

@jody jody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run Checkstyle (ant check style) to verify that your code modifications are consistent with the project's coding conventions.

Copy link
Contributor

@jody jody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please show that you have run checkstyle and there are no errors/warnings.

Copy link
Contributor

@jody jody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see request for evidence of having run checkstyle checks.

updating my repository
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
First try to get my code compliant with ant checkstyle

Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Second try

Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
Signed-off-by: CamCranda11 <159499223+CamCranda11@users.noreply.github.com>
@jody jody moved this from Ready to In progress in Console Game Hub Oct 24, 2025
@jody jody added the question Further information is requested label Oct 24, 2025
@jody
Copy link
Contributor

jody commented Oct 24, 2025

Note also: This PR does not indicate whether it closes the associated Issue.
image

@jody
Copy link
Contributor

jody commented Oct 24, 2025

There are currently 23 commits for this one change to the application. Please squash to a single commit.

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

Labels

question Further information is requested

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

6 participants