Skip to content

[bugfix] Fix edge case in Kramdown converter#810

Closed
honzasterba wants to merge 1 commit intogettalong:masterfrom
honzasterba:bugfix/fix_converter_li_p_edge_case
Closed

[bugfix] Fix edge case in Kramdown converter#810
honzasterba wants to merge 1 commit intogettalong:masterfrom
honzasterba:bugfix/fix_converter_li_p_edge_case

Conversation

@honzasterba
Copy link
Copy Markdown

weird combination of LI and P tags can cause the converter to crash

fixes #809

weird combination of LI and P tags can cause the converter to crash

fixes gettalong#809
@gettalong gettalong self-assigned this Nov 16, 2024
@gettalong gettalong added the bug label Nov 16, 2024
@gettalong
Copy link
Copy Markdown
Owner

Thanks for the pull request! The fix was not fully correct so I implemented the fix and test case myself.

@gettalong gettalong closed this Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] Kramdown converter fails with NoMethodError: undefined method `type' for nil

2 participants