Skip to content

PR #22 の際の文字コード変換時の誤りを修正 #51

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

Merged
merged 2 commits into from
Dec 7, 2019

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Dec 7, 2019

PR #22 の際の文字コード変換時の誤りを修正 (#47 の準備)

sakura-editor/sakura@e5172cb 時点での sakura 本体のヘルプを正にして
文字コード変換による問題のみを取り込んで修正する。

※ ver 2.4.0.0 のための修正は入っていません。あくまでも PR #22 の不具合修正です。

e5172cb526f26ad18ec5754282886b6270459588 時点での sakura 本体のヘルプを正にして
文字コード変換による問題のみを取り込んで修正する。
@m-tmatma
Copy link
Member Author

m-tmatma commented Dec 7, 2019

Code Factor の問題は sakura-editor/sakura#1111 をマージしてから
別 PR で修正する予定

Code Factor の警告対応
@m-tmatma
Copy link
Member Author

m-tmatma commented Dec 7, 2019

#530a6e7f9 を cherry-pick したら Code Factor の警告が消えました。

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いいんじゃないかと思います。

Shift_JIS ⇒ UTF-8 の誤変換以外はなんで変えてたのか謎ですが、
「リポジトリの特定れビジョンに合わせます」という内容で
「なんで」と訊くのも変な話だと思いますんで 😃

@m-tmatma m-tmatma merged commit c903df5 into sakura-editor:master Dec 7, 2019
@m-tmatma m-tmatma deleted the feature/fix-PR22-convertion branch December 7, 2019 11:43
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