You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Although with the current version that always removes all whitespaces it might not be a problem, for correctness, I think it would be better to also test Windows specific cases too.
This means doubling all tests that use \n with similar ones using \r\n.
In my fork that removes only one line terminator, the test file looks like this:
Although with the current version that always removes all whitespaces it might not be a problem, for correctness, I think it would be better to also test Windows specific cases too.
This means doubling all tests that use
\n
with similar ones using\r\n
.In my fork that removes only one line terminator, the test file looks like this:
The adjusted parser that passes these tests is:
The text was updated successfully, but these errors were encountered: