Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix legacy #5

Open
m1rza-s opened this issue Apr 20, 2018 · 1 comment
Open

Fix legacy #5

m1rza-s opened this issue Apr 20, 2018 · 1 comment
Assignees
@m1rza-s
Labels
cleanup
Projects
CORE

Comments

@m1rza-s
Copy link
Owner

m1rza-s commented Apr 20, 2018

Describe the issue:

Most of the code was written in 2016 by a student (me).

Propose a solution:

Use your current java-skill to write better code.
@m1rza-s m1rza-s self-assigned this Apr 20, 2018
@m1rza-s m1rza-s added this to To do in CORE via automation Apr 20, 2018
@m1rza-s m1rza-s moved this from To do to In progress in CORE Apr 20, 2018
@m1rza-s
Copy link
Owner Author

m1rza-s commented Apr 24, 2018

Finished fixing legacy issues in Monitor.java, Engine.java as well as Cell.java and all it's subclasses!
I uncovered some sneaky bugs along the way. Most of them got solved, but very few still lurk. :)

@m1rza-s m1rza-s moved this from In progress to Almost done in CORE May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m1rza-s m1rza-s

Labels
cleanup
Projects
CORE
  
Almost done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@m1rza-s