Skip to content

[Core] Use Parsedown for Markdown parsing#1783

Merged
em92 merged 1 commit intoRSS-Bridge:masterfrom
corenting:markdown_parsing
Oct 9, 2020
Merged

[Core] Use Parsedown for Markdown parsing#1783
em92 merged 1 commit intoRSS-Bridge:masterfrom
corenting:markdown_parsing

Conversation

@corenting
Copy link
Contributor

The RainbowSixSiege and NyaaTorrents website used a common snippet to parse markdown using some regex. This resulted in some errors during the parsing of some contents.

This PR aims to replace this custom code with the Parsedown. I had to change the bridges a little bit as the new parser had troubles with HTML escape characters in NyaaTorrents and with line breaks in RainbowSixSiege.

@em92
Copy link
Contributor

em92 commented Oct 7, 2020

Hi, @corenting !
Looks like parsedown library itself is missing in your PR

@corenting
Copy link
Contributor Author

Hi, @corenting !
Looks like parsedown library itself is missing in your PR

Oops, I just repushed the branch with the correct files.

@em92 em92 merged commit ff98efe into RSS-Bridge:master Oct 9, 2020
@em92
Copy link
Contributor

em92 commented Oct 9, 2020

gj @corenting !

@corenting corenting deleted the markdown_parsing branch October 12, 2020 07:30
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

Successfully merging this pull request may close these issues.

2 participants