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.

Sign up for GitHub

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

Jump to bottom

Add hasWhitespace and hasWhitespaceOnly #272

Merged

asaskevich merged 1 commit into asaskevich:master from jasonlam604:feature/add_hasWhitespace_and_hasWhitespaceOnly

Jul 20, 2018

Contributor

jasonlam604 commented Apr 13, 2018

Refer to issue #247

Although, I suggested IsBlank that that can get confusing with IsNull and in other languages isEmpty, it is debatably not completely clear.

In my opinion,

HasWhitespace is more explicit, does it have any whitespace
and
HasWhitespaceOnly is more explicit as well, does it only have whitespace

Included unit tests as well.


          Add hasWhitespace and hasWhitespaceOnly

54df467

asaskevich merged commit 4cba60e into asaskevich:master

Owner

asaskevich commented Jul 20, 2018

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment