-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
Added floating text program #89
Conversation
please see the codacy PR reviews. There's some issue on the os.system("cls") |
Okay Thank You. |
No need to send a PR again, You can edit that in this PR only |
All the checks are passed. |
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.
See review
"* ", | ||
" *** ", | ||
" *", | ||
"**** " ], |
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.
What about numbers ?
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.
And remove the cls not found command.
Its obstructing the view
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.
The program is just for English alphabets.
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.
Okay, I will fix that.
The error in the gif is fixed. Please review it again. |
Can you review my PR #89 please. |
Thank you for your contribution. Please provide the details requested below.
ISSUE NUMBER
#41
#20
SHORT DESCRIPTION
Floating Text is a simple program which displays a given input String as a floating string on the terminal.
TESTING
cd floating_text
python floating_text.py
Added a gif for your reference.