fix(MD013): preserve nested tables during list reflow#603
fix(MD013): preserve nested tables during list reflow#603mattiasgronlund wants to merge 4 commits intorvben:mainfrom
Conversation
|
Thanks for digging into this. Two things I'd like to flag before merging: 1. The regression test doesn't catch the bug. I reverted only the A different input does reproduce the underlying bug on - A long sentence about something interesting that describes how pipes appear in command-line examples.
cell a | cell bOn With this PR, it stays on its own line. So the fix is real, but the current test is a smoke test against a case that already passes on Per the project's testing philosophy, could you replace or supplement the test with a reproducer that genuinely fails before the fix? 2. Fix placement. The change is at line 1693 in the The Happy to chat through the design if helpful. |
53ccbac to
f88aa65
Compare
- A long sentence about something interesting that describes how pipes appear in command-line examples.
cell a | cell bIs not a nested table, I think this issue was actually fixed in 0.1.85, but I will try to understand the above issue, and provide a new pull request for that one. |
Description
Fixes MD013 semantic reflow so potential table rows nested inside list items are treated as structural lines, like fenced code markers. This prevents the fixer from merging nested table rows into paragraph text.
The regression test and bug fix were created in cooperation with Codex.
Fixes: #598
Type of Change
Testing
make test-dev)Added regression test:
test_reflow_preserves_tables_nested_in_list_itemsValidated locally:
Checklist
make fmt&&make lint)