Updated docstrings (now with module docstrings), comments, some variable names (or in one case, a constant), and added TypeHints!
Standard language for "no args no return" is not settled, may still need changing (see issue #67 Standardizing "no arg no return" language). If changes are made, ensure the following methods are updated:
updateBoard
checkBoard
resetGame
updatePlayerIcons
advancedGameSettings
gameSettingsPrompt
terminalGame
displayBoard
displayResult
Places where I got distracted and changed more than just documentation:
A tiny change in botMove, implemented a very smart suggestion from @MicahWW of replacing turn_counter with len(move_history)
typos on "traditionally" that have somehow persisted through like three different merges, BE FIXED TYPOS
removed extraneous checks in test_updateBoard_updates_board_correctly (was cycling through all printable characters to no effect after last update)
Hey, only three is not bad for me! (Yeah, yeah, I'm working on not changing things outside the scope of a given assignment...)
Updated docstrings (now with module docstrings), comments, some variable names (or in one case, a constant), and added TypeHints!
Standard language for "no args no return" is not settled, may still need changing (see issue #67 Standardizing "no arg no return" language). If changes are made, ensure the following methods are updated:
Places where I got distracted and changed more than just documentation:
Hey, only three is not bad for me! (Yeah, yeah, I'm working on not changing things outside the scope of a given assignment...)