Skip to content

consider whitespace between texts as text#7

Merged
navinre merged 1 commit intonavinre:masterfrom
amleshkashyap1:master
Jan 19, 2023
Merged

consider whitespace between texts as text#7
navinre merged 1 commit intonavinre:masterfrom
amleshkashyap1:master

Conversation

@amleshkashyap1
Copy link
Copy Markdown
Contributor

Consider whitespace between texts as text.

@amleshkashyap1 amleshkashyap1 marked this pull request as ready for review September 13, 2022 13:24
@sonal-amagi
Copy link
Copy Markdown

sonal-amagi commented Jan 18, 2023

We have a case where whitespace is present as text but the current workflow is considering it as the end of the text. line.strip! removes all the invisible characters and set line.empty? to be true

Screenshot 2023-01-18 at 1 20 48 PM

By using line.chomp! before line.strip! the white space can be considered as a text.

Screenshot 2023-01-18 at 1 30 43 PM

@navinre navinre merged commit 8ced1ef into navinre:master Jan 19, 2023
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.

3 participants