search

tbroadley / spellchecker-cli

A command-line tool for spellchecking files.
MIT License
119 stars 16 forks source link

Test fails on Windows(?) #106

Closed tbroadley closed 1 year ago

tbroadley commented 1 year ago

image

a2937 commented 1 year ago

Anything I can do to help further investigate the issue that occurred on my end?

tbroadley commented 1 year ago

@a2937 If you feel like investigating this yourself, please go ahead. Otherwise I can reproduce it locally, so I should be able to debug it.

a2937 commented 1 year ago

This is the problematic assertion.

https://github.com/tbroadley/spellchecker-cli/blob/0bfe195af57f17c9487315978c2cc8f403d810b9/test/cli-test.ts#L351

For some reason it doesn't recognize that the words wasn't repeated if they're on separate lines.

a2937 commented 1 year ago

This is the fixture in question.

https://github.com/tbroadley/spellchecker-cli/blob/master/test/fixtures/repeated-words.md

In GitHub it displays as one line but in reality it's two and what is causing the issue.

a2937 commented 1 year ago

Guessing this will be closed soon because this got resolved.

https://github.com/wooorm/parse-latin/commit/ebf6961ab3b961699c498e9416169b5e4d698e27

tbroadley commented 1 year ago

Nice, thanks for digging in and filing https://github.com/retextjs/retext-repeated-words/issues/10!