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

Investigate invalid \n|$ regex at end of parser's patterns #1880

Open
rafalp opened this issue Feb 22, 2025 · 0 comments · May be fixed by #1882 or #1901
Open

Investigate invalid \n|$ regex at end of parser's patterns #1880

rafalp opened this issue Feb 22, 2025 · 0 comments · May be fixed by #1882 or #1901
Assignees
@rafalp
Labels
area: backend This issue involves Python, Django or dependency (eg. database) feature: parser Parser converts markdown/bbcode strings into other forms
Milestone
Next feature release

Comments

@rafalp
Copy link
Owner

rafalp commented Feb 22, 2025 •
edited
Loading

\n|$ at the end of pattern will make regex match and consume linebreak at the end of, it preventing ^|\n at the beginning of next pattern to match with it. I need to check all those blocks patterns and make them focus on matching prefix linebreaks only and use look-ahead ((?=\n|$)) at the end.
@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 22, 2025
@rafalp rafalp added this to the Next feature release milestone Feb 22, 2025
@rafalp rafalp self-assigned this Feb 22, 2025
@rafalp rafalp linked a pull request Feb 23, 2025 that will close this issue
Open
6 tasks
@rafalp rafalp linked a pull request Mar 15, 2025 that will close this issue
Open
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
1 participant
@rafalp