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 Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert して CI ビルドが通るようにする #940

Merged
merged 2 commits into from
Jun 2, 2019
Merged

HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert して CI ビルドが通るようにする #940

merged 2 commits into from
Jun 2, 2019

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Jun 1, 2019 •
edited
Loading

PR の目的

HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert して CI ビルドが通るようにする

カテゴリ

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

PR の背景

#937 と #939 を入れたが、
https://dev.azure.com/sakuraeditor/sakura/_build/results?buildId=543
でビルドに失敗している。

ただ https://dev.azure.com/sakuraeditor/sakura/_build/results?buildId=544 は成功している。
今日試したときに何度か、ローカルビルドでもビルドに失敗した。
何か失敗の条件があると思うが、わかっていない。

PR のメリット

CI でビルドが通るようになる。

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

モチベーションが下がる。

PR の影響範囲

Compiled HTML Help (chm)

関連チケット

#906
#937
#939

参考資料

@m-tmatma
Revert "Merge pull request sakura-editor#939 from m-tmatma/feature/fi…
f58e67a 
…x-js-utf8"

This reverts commit d266e8c, reversing
changes made to 9733cc5.
@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 changed the title HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert する HTML Help 用の HTML ファイル、Javascript の UTF-8 対応を revert して CI ビルドが通るようにする Jun 1, 2019
@m-tmatma m-tmatma mentioned this pull request Jun 1, 2019
Merged
@m-tmatma m-tmatma marked this pull request as ready for review June 1, 2019 23:01
@m-tmatma m-tmatma changed the title HTML Help 用の HTML ファイル、Javascript の UTF-8 対応を revert して CI ビルドが通るようにする HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert する Jun 1, 2019
@m-tmatma m-tmatma changed the title HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert する HTML Help 用の HTML ファイル、Javascript の UTF-8 対応 (#937 と #939) を revert して CI ビルドが通るようにする Jun 1, 2019
@m-tmatma m-tmatma added this to the v2.4.0 milestone Jun 1, 2019
@m-tmatma
Copy link
Member Author

m-tmatma commented Jun 1, 2019

ビルド通りました。

@m-tmatma m-tmatma merged commit e2a2df5 into sakura-editor:master Jun 2, 2019
This was referenced Jun 2, 2019
Merged
Merged
Closed
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 m-tmatma deleted the feature/revert-utf8-chm branch July 5, 2019 21:55
m-tmatma added a commit to m-tmatma/sakura that referenced this pull request Jul 6, 2019
@m-tmatma
Revert "Merge pull request sakura-editor#940 from m-tmatma/feature/re…
aec381e 
…vert-utf8-chm"

This reverts commit e2a2df5, reversing
changes made to d266e8c.
This was referenced Jul 25, 2019
Closed
Merged
Closed
@berryzplus berryzplus mentioned this pull request Aug 16, 2019
Merged
@KENCHjp KENCHjp added CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント labels Jan 7, 2020
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
CI appveyor など CI 関連 【ChangeLog除外】 document ドキュメント
Projects
None yet
Milestone
v2.4.0
Development

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