snoopdave commented on PR #228: URL: https://github.com/apache/roller/pull/228#issuecomment-6453925006
**🤖 GLM: PR-Review — General Issues** The following issues were found but cannot be attached to a specific line in the diff: - **Important:** CHANGES.md has no entry for this change. Other PRs in the 6.1.7 batch each carry one. Worth a note: the main menu now prints the weblog description as plain text (previously it was inserted as markup), and `removeHTML` drops an unterminated tag rather than keeping it. Strengths (not action items): `startsTag` matches browser tokenization closely (letter, `/`, `!`, `?` after the `<`), the two new tests cover both the dropped-tag and kept-text cases, and dropping `escapeHtml="false"` is the right call given About is plain text once `removeHTML` has run. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
