-
Notifications
You must be signed in to change notification settings - Fork 174
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.
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
Readme.txt changelog parsing issue after replacing Markdown with Parsedown #94
Comments
Thank you for the report, I hadn't noticed this regression. It should probably be fixed now. Switching to another parser is an option, but it could be a time-consuming task. I would prefer to stick to the current implementation unless it turns out to have some hard-to-fix issues. |
Thank you for the quick update. Works for me. I'm thinking about adding tests to prevent regressions and change minor things to make sure the response is the same to wp.org API. For now, it looks for me like the ideal way is to keep it exactly the same, byte-to-byte. I understand that a few more or less spaces in the HTML parts, for example, won't hurt anyone. But is this assumption correct in general? For example, won't it break something in the plugin-update-checker counterpart? Do you see any consequences or obstacles going this way? |
In general, this server should produce the same results as wp.org when parsing plugin readmes. Or at least a subset of those results, as this implementation doesn't handle screenshots and possibly a few other things. However, the update API responses are intentionally different from the wp.org API. Here's why:
|
Ok, got it. |
After updating to the latest version in master, I have noticed a plugin changelog is shown incorrectly in the WordPress dashboard.
A regression introduced in this commit: 659be8d
Input:
Output before:
Output after:
I have found that a "native" wp.org readme parser is available. Wondering if it would be the best option to stick with to keep the compatibility?
The text was updated successfully, but these errors were encountered: