Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MD032: add an option to forbid lazy continuation #979

Open
papperlapapp opened this issue Feb 3, 2024 · 4 comments
Open

MD032: add an option to forbid lazy continuation #979

papperlapapp opened this issue Feb 3, 2024 · 4 comments

Comments

@papperlapapp
Copy link

The CommonMark spec allows lazy continuation lines in lists, e.g.

1.  A paragraph
with two lines.

and pymarkdown is currently happy with these.

I'd rather prefer to have an option to disable this, e.g. require my users to write the above as

1.  A paragraph
    with two lines.
@jackdewinter
Copy link
Owner

This is probably doable as a rule, any kind of fix support would take longer. Does that work for you?

@jackdewinter jackdewinter added the requires more information There is not enough information to make progress. label Mar 19, 2024
@papperlapapp
Copy link
Author

papperlapapp commented Mar 24, 2024

I'm not sure I understand your question. Adding an option to the existing rule just appeared natural to me. From a user's perspective, I don't see a difference between an option to an existing rule or a separate rule. The results of both would be identical.

Interestingly (and FWIW), the MD032 rule in https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md032---lists-should-be-surrounded-by-blank-lines forbids lazy continuation lines.

@jackdewinter
Copy link
Owner

Interesting... its has changed since I last look at it.

I am leaning towards a new rule, as it is clearer to the user. Reading that rule, it seems to be more of a "side effect" instead of an intended behavior. Its one the list... it should be relatively easy to implement and test.

@jackdewinter jackdewinter removed the requires more information There is not enough information to make progress. label Mar 30, 2024
@jackdewinter
Copy link
Owner

Btw, just coming across a lot of these in our current testing of triply nested containers. Thinking that we might have time to work on this before the end of this next month. Stay tuned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants