-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feature/clean screen function #98
Feature/clean screen function #98
Conversation
- Added doctest for clear_screen function to verify it runs without errors.
- Develop clear_screen function to clear the console screen.
- Updated __main__.py to call clear_screen function when reset mode activated
I noticed the os.system usage in the code. If necessary, I can update it to use the subprocess module to address the security concern. Just let me know if you’d like me to make this change. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #98 +/- ##
==========================================
- Coverage 97.47% 96.54% -0.92%
==========================================
Files 2 2
Lines 197 202 +5
Branches 22 23 +1
==========================================
+ Hits 192 195 +3
- Misses 4 5 +1
- Partials 1 2 +1 ☔ View full report in Codecov by Sentry. |
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.
LGTM 🔥
@oscarArismendi Thanks for your contribution 🚀 |
Reference Issues/PRs
What does this implement/fix? Explain your changes.
This pull request integrates the
clear_screen
function into the main module (__main__.py
). Specifically, the function is called when the reset mode is activated. This enhancement ensures that the console screen is cleared appropriately, providing a cleaner interface during reset operations.Any other comments?
For additional context, you can view the related GIF below: