Skip to content

Escape HTML tags in commit titles #43

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
silverwind opened this issue Dec 6, 2016 · 0 comments
Closed

Escape HTML tags in commit titles #43

silverwind opened this issue Dec 6, 2016 · 0 comments

Comments

@silverwind
Copy link

silverwind commented Dec 6, 2016 •
edited
Loading

See nodejs/node#10127 (comment). A HTML tag in a commit message can throw off markdown parsers. I think problematic chars like < should be escaped as \<.
@silverwind silverwind mentioned this issue Dec 6, 2016
Merged
2 tasks
rvagg added a commit that referenced this issue Feb 22, 2017
@rvagg
escape <> in markdown
01077e4 
Fixes: #43
@rvagg rvagg mentioned this issue Feb 22, 2017
Merged
@rvagg rvagg closed this as completed in #44 Feb 23, 2017
rvagg added a commit that referenced this issue Feb 23, 2017
@rvagg
escape <> in markdown
87c2ebf 
Fixes: #43
renawolford6 added a commit to renawolford6/changelog-maker-javascript that referenced this issue Nov 9, 2022
@renawolford6
escape <> in markdown
d7c9680 
Fixes: nodejs/changelog-maker#43
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@silverwind