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.

Sign up for GitHub

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

Jump to bottom

Fix invalid parser's blocks patterns #1882

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Fix invalid parser's blocks patterns #1882

wants to merge 6 commits into from
+389 −15

Conversation

rafalp
Copy link
Owner

@rafalp rafalp commented Feb 23, 2025 •
edited
Loading

Adds test checking that block next to other block is parsed as two separate blocks, with extra logic for handling exceptions.

Also fixes lists parser because that one is very out of spec.

Fixes #1880
Fixes #1884

TODO

  • Fix lists regexes
  • Add missing blocks to the test matrix

@rafalp rafalp added area: backend This issue involves Python, Django or dependency (eg. database) feature: parser Parser converts markdown/bbcode strings into other forms labels Feb 23, 2025
@rafalp rafalp added this to the Next feature release milestone Feb 23, 2025
@rafalp rafalp self-assigned this Feb 23, 2025
@rafalp rafalp force-pushed the fix-1880-cleanup-linebreak-in-patterns branch from 04046e0 to 842e1bc Compare February 25, 2025 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@rafalp rafalp

Labels
area: backend This issue involves Python, Django or dependency (eg. database) feature: parser Parser converts markdown/bbcode strings into other forms
Projects
None yet
Milestone
Next feature release
Development

Successfully merging this pull request may close these issues.

Support multiline list items Investigate invalid \n|$ regex at end of parser's patterns
1 participant
@rafalp