-
Notifications
You must be signed in to change notification settings - Fork 0
Simplify codebase and add ASCII art timer display #2
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
base: main
Are you sure you want to change the base?
Conversation
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.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ncement Co-authored-by: vysor <10956296+vysor@users.noreply.github.com>
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.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: vysor <10956296+vysor@users.noreply.github.com>
… clearing Co-authored-by: vysor <10956296+vysor@users.noreply.github.com>
…dling, remove redundant check Co-authored-by: vysor <10956296+vysor@users.noreply.github.com>
Co-authored-by: vysor <10956296+vysor@users.noreply.github.com>
Consolidated the timer implementation from 2 files (1,891 lines) to a single 280-line module with ASCII art visualization.
Changes
Simplified architecture
Menu.pyclass logic directly intotimer.py.gitignorefor Python artifactsASCII art display
[████████░░░░] 65%Code quality
Example
The timer now displays:
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.