make list-item-spacing's message more clear

[despairblue]

fix #60

[wooorm]

Looks clean, simple, and good! Was there anything you struggled with, or other suggestions for this project?

[despairblue]

Nope. Your comment told me what rule is responsible for that test. After that it was just

1. finding the correct file (easy because the file names are the rule names)
2. finding the the part that creates the warning
3. understanding a little bit more about the file (very clean, understandable code 👍 )
4. changing the output.

So maybe the linter could prepend error/warning with the rule's name (like eslint does). I'm not sure if it already does that in a terminal, I only interacted with it through Atom.

[wooorm]

Great, thanks for the feedback!

What you’re talking about is called a ruleId (both in ESLint and in VFile’s, which are used here), and those are populated by the main remark-lint library. The standard reporter displays those at the end of a message.

As you’re using Atom, displaying the ruleId should be enabled in that library. If you’re using linter-markdown, it should be supported already, though. If not, please raise an issue. Or, if using another library, raise an issue over there.

Does that help?

[despairblue]

You're right, I must have overseen it. My bad. Then everything is alright from my side.

[despairblue]

On another note, why is it necessary that lists containing wrapped items must be loose? That adds a p and br tags to each item in that list and thus changes the rendering, at least the github renderer does it.

This seems to make it impossible to have a free flowing list item while still adhering to the 80 character line limit.

Example

• Wrapped item Dolor quas fugiat saepe qui. Aperiam eveniet autem ab et ab
  temporibus vero quo. Dolor quas fugiat saepe qui. Aperiam eveniet autem ab
  et ab temporibus vero quo.
• item 2
• item 3

---

• Wrapped item Dolor quas fugiat saepe qui. Aperiam eveniet autem ab et ab
  temporibus vero quo. Dolor quas fugiat saepe qui. Aperiam eveniet autem ab
  et ab temporibus vero quo.
• item 2
• item 3

[wooorm]

Yes, that’s right. It’s from the markdown-style-guide, which is the source of inspiration and rules for this library. I personally prefer it too, as it makes reading the source easier, although indeed rendering on GitHub is less beautiful.

Note that you can always turn it off!

[wooorm]

Thanks @despairblue for working on this! 👍