Fails to detect single sentence spanning multiple lines

[andrewrynhard]

Fails to detect when a single sentence spans multiple lines:

$ cat test.md
# Test

This is a single sentence that spans
multiple lines.
$ markdownlint --rules ./node_modules/sentences-per-line/index.js test.md
$ echo $?
0
$

[JoshuaKGoldberg]

Ha, good find. I think it's reasonable to add this as an opt-in optional rule flag, as it's common practice to split long sentences across lines. How about the following options:

• true to only allow one sentence per line: no more, no less
• A number to allow a sentence to split across lines if its length is > that number?

[andrewrynhard]

Sounds reasonable to me!

[JoshuaKGoldberg]

@all-contributors please add @andrewrynhard for ideas.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙

[allcontributors]

@JoshuaKGoldberg

I've put up a pull request to add @andrewrynhard! 🎉

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.