Skip to content

fix utf8 issue + remove deprecated has_rdoc #54

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

Closed
wants to merge 2 commits into from
Closed

Conversation

ut
Copy link

@ut ut commented Nov 28, 2012

i fixed the utf8 issue (which occurs if non-latin1 chars are used in a git commit message) with a solution described in issue #33 and by the way i removed a deprecated syntax for rdoc

@dougo
Copy link

dougo commented Sep 9, 2013

This UTF-8 bug still occurs in 1.2.6. Any plans to merge this?

@jeffweiss
Copy link

I would also like to see this merged because it fixes an error for me because this commit puppetlabs/puppet@d27af92 accidentally snuck in a unicode nbsp in the log message.

@robertodecurnex robertodecurnex force-pushed the master branch 2 times, most recently from 3fc1c27 to b224195 Compare February 25, 2016 17:56
@perlun
Copy link
Contributor

perlun commented Jan 15, 2018

Thank @ut. Would you have a chance to rebase this on top of the current master? So we can see if it is CI clean etc.

@ut
Copy link
Author

ut commented Feb 2, 2018

Thanks @perlun. I checked my PR: The fix provided has been solved in the meantime by commit fd86ddd in reference to issue #190. I've verified the fix by using the example in #33, which worked for me. You might consider to close #33 and #190 also.

@perlun
Copy link
Contributor

perlun commented Feb 2, 2018

Alright, great! Thanks for the confirmation. Closing this now.

@perlun perlun closed this Feb 2, 2018
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.

4 participants