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

Escape HTML in commit titles #115

Merged
merged 4 commits into from
Jan 16, 2018
Merged

Escape HTML in commit titles #115

merged 4 commits into from
Jan 16, 2018

Conversation

hoodwink73
Copy link
Contributor

Do not escape HTML if the user is applying a custom hook on the changelogs

Fixes #91

@hoodwink73
Escape HTML in commit title while generating changelogs
fd81968 
Do not escape HTML if the user is applying a custom hook on the
changelogs
@hoodwink73
Fix lockfile
cdacc31
@hoodwink73
Copy link
Contributor Author

When I npm link-ed my local version of the repo, some dependencies were missing. So, I had to manually install them.

Before raising the PR, I uninstalled those dependencies leaving aside the new dependency, escape-goat.

But now the CI is complaining about missing dependencies.

@leo leo changed the title Escape HTML in commit title while generating changelogs Escape HTML in commit titles Jan 16, 2018
@leo leo self-requested a review January 16, 2018 10:16
@leo
Copy link
Contributor

leo commented Jan 16, 2018

Wonderful, thank you! 😊

@leo leo merged commit 2e66623 into vercel:master Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leo leo leo approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@hoodwink73 @leo