Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Trailing whitespaces trimmed with `Keep Markdown Line Break Whitespace` option

See original GitHub issue

Hi everyone, I know this has been a topic of discussion some months ago (see #115 and the FAQ), however it is not clear to me whether the package is in sync with the whitespace package option Keep Markdown Line Break Whitespace.

I have been discussing this with the whitespace maintainers here and from our test it seems that the problem is related with this package which does not “parse” the Keep Markdown Line Break Whitespace option. This new option should avoid users to add

'*':
  # all current config
'.md.text':
  whitespace:
    removeTrailingWhitespace: false

in the config.cson. Am I right?

Issue Analytics

State:
Created 6 years ago
Comments:7 (4 by maintainers)

Top GitHub Comments

2reactions

burodepepercommented, Oct 24, 2017

It is related to this issue on the whitespace package. It checks whether the document scope matches that set by Atom’s default language-gfm package, and not for the default scope from language-markdown.

I can make/submit the PR that’s suggested in that thread, but it will have to wait until I’m less busy with my normal work. If that PR is accepted, it should indeed remove the need for the setting you quoted.

1reaction

burodepepercommented, Sep 6, 2018

PR has been merged. I expect it to be part of the next update of Atom