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

Fixed a bug that the profile tab could not be opened when the source … #490

Merged
merged 1 commit into from
Aug 26, 2021
Merged

Fixed a bug that the profile tab could not be opened when the source … #490

merged 1 commit into from
Aug 26, 2021

Conversation

Taikono-Himazin
Copy link
Contributor

There was a bug that the profile tab could not be opened when the source code contained Japanese. The error message is as follows.

'cp932' codec can't decode byte 0x88 in position 19: illegal multibyte sequence

Obviously, it is due to a character code problem.
Basically, the source code should be written in UTF-8, so I explicitly specified to read it in utf-8.

…code contained Japanese.

There was a bug that the profile tab could not be opened when the source code contained Japanese. The error message is as follows.

'cp932' codec can't decode byte 0x88 in position 19: illegal multibyte sequence

Obviously, it is due to a character code problem.
Basically, the source code should be written in UTF-8, so I explicitly specified to read it in utf-8.
@codecov
Copy link

codecov bot commented Aug 25, 2021

Codecov Report

Merging #490 (0650fab) into master (7f03cfe) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   82.83%   82.83%           
=======================================
  Files          50       50           
  Lines        2051     2051           
=======================================
  Hits         1699     1699           
  Misses        352      352           
Impacted Files Coverage Δ
silk/views/code.py 87.87% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f03cfe...0650fab. Read the comment docs.

@Archmonger
Copy link
Contributor

Great addition, thanks!

@Archmonger Archmonger merged commit 3e8f443 into jazzband:master Aug 26, 2021
@Taikono-Himazin Taikono-Himazin deleted the patch-1 branch August 26, 2021 12:05
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