Skip to content
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

Please add "twinkle_colors" and "upLinePrompt" functions to colorTest.cpp. #94

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shin4523
Copy link

I tried running colorTest.cpp in visual studio. By the way, the output screen went too fast and it was hard to see at a glance. "rang" function and output screen did not match well. So we created a "twinkle_colors" function that makes certain text twinkle so that we can easily see the color change. I used the rand and srand functions. I also used the upLinePrompt function, which allows you to go to a specific location on the console screen and re-print the text.

Add "twinkle_colors" and "upLinePrompt" functions.
add "twinkle_colors" and "upLinePrompt" functions to colorTest.cpp.
The pull request did not work properly and I deleted the comment.
I think it's because of the comment.
@agauniyal
Copy link
Owner

Hey @shin4523 I'm bit busy at the moment but there's another version of rang library in the pipeline, so I'll merge this PR with that version since the tests are gonna change slightly. Thanks for your work!

@shin4523
Copy link
Author

Oh, thank you! Could you tell me when the new version will be available? And do you mean that my code will be a merge in that version?
I am sorry that I could not understand your comment properly because I am not good at English.

@agauniyal
Copy link
Owner

@shin4523 I can't promise a date, there are only few things pending for next version so as soon as I get some time to complete them I'll merge your code in that version 😄

@shin4523
Copy link
Author

Okay. Thank you very much!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants