JoustMania icon indicating copy to clipboard operation
JoustMania copied to clipboard

Published 20 hours ago • verified publisher icon adangert

Large Refactor

Open mchirlin opened this issue 1 year ago • 2 comments

Refactoring

  • Moving all games to use a common class: games.game
  • Reusing as much code as possible across games
  • Added logging to file throughout
  • Removed a lot of duplicate type variables - moved all game_opts to a single variable

Bug Fixes

  • #314

Enhancements

  • Adding end game sounds to all of the games
  • #286
  • #281
  • #179
  • #162
  • #19
  • #16

mchirlin avatar Feb 05 '24 15:02 mchirlin

Thanks! Next time that I have some spare time, I will test out the code changes, are you using bookworm or bullseye for the raspberry pi OS btw?

adangert avatar Feb 07 '24 14:02 adangert

I think it's bullseye.

mchirlin avatar Feb 07 '24 16:02 mchirlin

Fantastic work!! Just went through the code, and was able to test all the game modes, There were a couple of bug fixes that I was able to find and update as well. But all the new code is much cleaner with way less DRY code everywhere. Thanks so much for all the help!

adangert avatar Mar 11 '24 04:03 adangert