-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Maze generation and integration with fish npc #379
Conversation
…aze file to ensure it works.
…he main game. Fixed code modulaity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work. Unit tests could be more comprehensive though but besides that good job!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great implementation of the feature! Good testing as well and well-structured code with ample javadocs! Ensure documentation is thoroughly filled as well. Good stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, definitely NEED more/more thorough documentation throughout but you guys have time to fix that up and code coverage isn't amazing but for base functionality it works. Again just working on breaking up some methods. Other than that it's good!
This pull request integrates the new underwater maze mini-game with the fish npc.
The current game randomises the maze design and we have implemented a light radius (controlled with wads or arrows).
Screen should resize appropreiately. Additionally, a return to game button for the end game screen has been implemented when the mini-game is played from the main game.
Extensive testing and further game development is needed but integration with main game and maze design is complete. (Do note that some files will have a lot more work done specifically MazeGameScreen and MazeGame)
Feature: Underwater Maze Mini-Game #333
Task Tickets:
Underwater Maze Design #378
Integration of Underwater Maze Mini-Game with Fish NPC #377