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.

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

Fix UnicodeEncodeError #4

Closed
wants to merge 0 commits into from
Closed

Fix UnicodeEncodeError #4

wants to merge 0 commits into from

Conversation

skull-squadron
Copy link
Contributor

Before

0 ~/Projects/homepage/static git:master ❯❯❯ starred --username steakknife --sort > README.md

Traceback (most recent call last):
  File "/usr/local/bin/starred", line 11, in <module>
    sys.exit(starred())
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/starred.py", line 44, in starred
    data = '* [{}]({}) - {}'.format(*repo)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xae' in position 33: ordinal not in range(128)
1 ~/Projects/homepage/static git:master ❯❯❯ 

After

0 ~/Projects/homepage/static git:master ❯❯❯ starred --username steakknife --sort > README.md

0 ~/Projects/tr/static git:master ❯❯❯ wc README.md
    1163   12119  120433 README.md
0 ~/Projects/homepage/static git:master ❯❯❯

@xiaopeng163
Copy link
Contributor

There have the same problem for repo description.....

@skull-squadron
Copy link
Contributor Author

skull-squadron commented Jul 9, 2016

@xiaopeng163 Sorry, I don't understand what you're trying to say.

@xiaopeng163
Copy link
Contributor

sorry, my fault. I missed the commit of 570f884

@skull-squadron
Copy link
Contributor Author

skull-squadron commented Jul 9, 2016

There's a new, similar bug, new PR incoming.

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.

None yet

2 participants