-
-
Notifications
You must be signed in to change notification settings - Fork 531
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
avoid to add empty space between words and punctations. #178
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.
Can you please add the test cases for this fix? Thank you
I have never seen extra spaces added, do you have example? |
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.
Please, check my comments. I am what is the critical part in the test.
Hi, any problem with the merge ? |
Yes, sorry, the time is my problem. I hope I get to this soon. |
Ok, no problem, thanks! |
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.
Thank you for your fix and your patience with me 🙂
This small fix, avoid to add empty space between words and punctation symbols. I realized that there were annoying spaces in the final text.