Skip to content

Comments added to existing files#963

Merged
jadeandtea merged 14 commits intoBram-Hub:devfrom
Agnewkidjr:dev
Feb 24, 2026
Merged

Comments added to existing files#963
jadeandtea merged 14 commits intoBram-Hub:devfrom
Agnewkidjr:dev

Conversation

@Agnewkidjr
Copy link
Collaborator

Description

Added comments for Sudoku, Minesweeper, and Fillapix. The comments were at the top of the functions that did not have any. Fixed some grammatical errors that where in some comments.
Added a link to the set up tutorials to the readme.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (improvement to an already existing feature)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

N/A did not add code only comments.

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

main thing is adding be in front of overridden so that the rest of the sentence makes sense when talking about overriding methods.
Did almost all the comment in the possible Number Case Board class
added comments to the model Sudoku boards class
added comments to the Sudoku board class as well as Element View class
-Minesweeper drawing of the elements commented
-Sudoku cell corrected wording
pp
-Cell factory minor grammar fix
- Sudoku view commented on board dimensions
Added links to tutorials for Proof Editor and Puzzle Editor to the student section
This reverts commit 3f42ff8, reversing
changes made to 5336081.
Added comment to Minesweeper Utilities  for every function except for Combination and recursive combination
Added back the links for the tutorials
ADDed comments to Importer data Tile data and fixed grammer in Utilites
-Fillapix Utilities
- Fillapix Cell
fixed gramamer in
-Fillapix Cell Factory
-Fillapix
Copy link
Collaborator

@jadeandtea jadeandtea left a comment

Choose a reason for hiding this comment

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

Looks good

@jadeandtea jadeandtea merged commit fad6207 into Bram-Hub:dev Feb 24, 2026
9 checks passed
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.

2 participants