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.

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

Refactor #23

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Refactor #23

wants to merge 4 commits into from

Conversation

fardog
Copy link
Owner

@fardog fardog commented Jan 5, 2022

Some new features will be added to Anton in the future; namely:

  • additional reporters
  • some sort of display
  • better web UI

However the exiting main.cpp was difficult to navigate, and had many concerns mixed up in it. This separates out a few, imperfectly. There are still too many concerns in each and they are too tightly coupled, but it's still much easier to reason about.

these various classes are still really interdependent, but at least
they begin to cordon off a few things in preparation for adding a
display
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.

1 participant