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

HTML ヘルプの文字コードを UTF-8 BOM 付きに変換することにより、Web サイトのデータとヘルプのデータを簡単に同期できるようにする #937

Merged
merged 5 commits into from
Jun 1, 2019
Merged

HTML ヘルプの文字コードを UTF-8 BOM 付きに変換することにより、Web サイトのデータとヘルプのデータを簡単に同期できるようにする #937

merged 5 commits into from
Jun 1, 2019

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented May 31, 2019 •
edited
Loading

PR の目的

HTML ヘルプの文字コードを UTF-8 BOM 付きに変換することにより、Web サイトのデータとヘルプのデータを簡単に同期できるようにする。

カテゴリ

  • ドキュメント修正
  • CI関連
    • Appveyor
    • Azure Pipelines

PR の背景

  1. Web サイト でも同様のデータを公開している。
  2. Web サイト では文字コードを UTF-8 に変換している。
  3. 一方が UTF-8 でもう一方が ShiftJIS だと簡単に最新版を反映できない。
  4. ヘルプも UTF-8 に変換して管理しやすくする

#807 で実施しようとしていたが、 #906 の問題が発生したため、実施が延期になっていた。
#921 および #930 の対応を入れたことにより #906 の問題が解消したので UTF-8 対応を
行う準備が整った。

なお、 現在 #922 の問題は発生しているが、これはこのビルド環境が英語であることに
起因している問題なので、この PR とは無関係です。

PR のメリット

  • Web サイトのデータと簡単に同期がとれる
  • GitHub 上で差分をレビューしやすくなる。

PR のデメリット (トレードオフとかあれば)

現在はなし。
(#921 および #930 により CI 上でのビルドエラーの問題は解消した)

PR の影響範囲

HTML Help (chm ファイル)

関連チケット

#807
#906
#921
#930
#922 (これは影響しない)

参考資料

#807 (comment) の python スクリプトを使用して文字コード変換を行いました。

@m-tmatma m-tmatma added this to the v2.4.0 milestone May 31, 2019
@m-tmatma m-tmatma added appveyor azure pipelines CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント labels May 31, 2019
@m-tmatma m-tmatma mentioned this pull request May 31, 2019
Closed
@m-tmatma m-tmatma merged commit 9733cc5 into sakura-editor:master Jun 1, 2019
This was referenced Jun 1, 2019
Closed
Merged
m-tmatma added a commit to m-tmatma/sakura that referenced this pull request Jun 1, 2019
@m-tmatma
Revert "Merge pull request sakura-editor#937 from m-tmatma/feature/ht…
e9fde9f 
…mlhelp-utf8-take4"

This reverts commit 9733cc5, reversing
changes made to c0c4a19.
@m-tmatma m-tmatma mentioned this pull request Jun 1, 2019
Merged
m-tmatma added a commit that referenced this pull request Jun 2, 2019
@m-tmatma
Merge pull request #940 from m-tmatma/feature/revert-utf8-chm
e2a2df5 
HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert して CI ビルドが通るようにする
@m-tmatma m-tmatma deleted the feature/htmlhelp-utf8-take4 branch June 2, 2019 02:39
@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 2, 2019

#940 で revert

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
@m-tmatma
Merge pull request sakura-editor#937 from m-tmatma/feature/htmlhelp-u…
63dd732 
…tf8-take4

HTML ヘルプの文字コードを UTF-8 BOM 付きに変換することにより、Web サイトのデータとヘルプのデータを簡単に同期できるようにする
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
@m-tmatma
Revert "Merge pull request sakura-editor#937 from m-tmatma/feature/ht…
e226bf9 
…mlhelp-utf8-take4"

This reverts commit 9733cc5, reversing
changes made to c0c4a19.
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
@m-tmatma
Merge pull request sakura-editor#940 from m-tmatma/feature/revert-utf…
18324af 
…8-chm

HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (sakura-editor#937 と sakura-editor#939) を revert して CI ビルドが通るようにする
m-tmatma added a commit that referenced this pull request Jul 26, 2019
@m-tmatma
Revert "HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を …
7bb4c3a 
…revert して CI ビルドが通るようにする"
m-tmatma added a commit that referenced this pull request Jul 28, 2019
@m-tmatma
Revert "HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を …
1b45775 
…revert して CI ビルドが通るようにする"
@m-tmatma m-tmatma mentioned this pull request Jul 28, 2019
Closed
berryzplus pushed a commit to berryzplus/sakura-editor that referenced this pull request Aug 4, 2019
@m-tmatma @berryzplus
Revert "HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (sakura-editor#937…
5502d6c 
… と sakura-editor#939) を revert して CI ビルドが通るようにする"
berryzplus pushed a commit to berryzplus/sakura-editor that referenced this pull request Aug 15, 2019
@m-tmatma @berryzplus
Revert "HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (sakura-editor#937…
c297948 
… と sakura-editor#939) を revert して CI ビルドが通るようにする"
@berryzplus berryzplus mentioned this pull request Aug 16, 2019
Merged
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
@m-tmatma @berryzplus
Revert "HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (sakura-editor#937…
8735a2c 
… と sakura-editor#939) を revert して CI ビルドが通るようにする"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@berryzplus berryzplus berryzplus approved these changes

Assignees
No one assigned
Labels
appveyor azure pipelines CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント
Projects
None yet
Milestone
v2.4.0
Development

Successfully merging this pull request may close these issues.
2 participants
@m-tmatma @berryzplus