-
Notifications
You must be signed in to change notification settings - Fork 22
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
Different changes, see PR #58 #60
Conversation
This reverts commit 262e0c6.
This is almost the same stuff as I had in #58 but without 4->2 indent (it can be done in a separate PR). I may have introduced some new bugs but I tried to not alter the program logic (but I simplified it a bit in some places) |
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.
This all looks good to me. A lot easier to read
This is an amazing PR, thank you so much @Yardanico! I'll get a few more PRs merged today and then cut a release 💪 Thanks @Tangdongle for the review too |
- Massive code cleanup, also fixes 1+1 error (lol) [#60] - Added help menu [#55] - Rewrite of input functionality, using `noise`, fixes c-d [56] Many thanks to @Tangdongle and @Yardanico
@Yardanico I believe this is causing #63 |
I merged without running |
haha no worries; it's all about setting up automation to remove human errors; no one should have to manually run |
Almost all of this is described in #58 , I tried to make separate commits.
I think that after reviewing it's better to do a squash merge so that this "revert" commit doesn't get into history