refactor: code rationalization - WIP #27
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi there, happy new year!
this is a (still dirty) refactor of the code to make it more readable and maintainable.
Feature highlight:
pydanticfor (json) settings managementpathlibtyperfor CLI design (I changed some options and organized them by commands)tkinter(Framesubclass) andpygame(SpriteandGroupsubclasses) windows and componentstkinterwindow to handle variables and pass the data to the related methodsFrame.mainloop()intkinterwindows instead of manually handle the loop.Still need to extensively test, add animations and build the
pyinstallerexecutables.I'm publishing it as a Draft because I don't think I'll have much time in the following weeks to continue;
I hope this all makes sense to you and that can be helpful to build the next iteration.