Bug: Carriage return causes infinite loop

[kiwiroy]

See gist for example.

npm install
npm test

README.md and example-good.md should be fine, example-bad.md is bad

Removing either .remarkrc or example-bad.md allows the test to pass. As does an in-place edit with perl.

perl -i -lne 's/\r//g; print' example-bad.md

[wooorm]

So, there’s two parts to this issue:

1. remark-lint enters infinite loop; I’ll push a fix for this;
2. Carriage returns are not supported. Fix being to strip them before passing to remark (most editors do that by default or are configurable to do it). Long term fix is to allow carriage returns everywhere where newlines are also possible in the parser, instead of removing them in a preprocessing step. The latter will be a lot of work and I’m not really interested in doing that. But if you’d like to take a stab, I’d gladly merge a PR!

[wooorm]

I’ll use this issue to refer to the first, if you’re interested in the second, could you open a new issue?