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

Just kidding #1

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

Conversation

programadorthi
Copy link

No description provided.

Thiago Santos and others added 4 commits November 29, 2019 15:58
Improved translator class to use built-in kotlin operators

Added check to valid character in the phrase to avoid strange behaviors

Added unit test to check valid characters in the phrase
Improved avoid to execute search when the character is whitespace

Added some comments (Yes, this is not a best practices, I know)
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.

1 participant