appveyor で文字コードをチェックして UTF-8 になっていなかったら処理を失敗させる

[m-tmatma]

#313: appveyor で文字コードをチェックして UTF-8 になっていなかったら処理を失敗させる

cpp/h ファイルの文字コードをチェックして、UTF-8 あるいは ascii で
なかったら appveyor ビルドを失敗させる

テスト的に shift-jiis で cpp ファイルを追加したもの
https://ci.appveyor.com/project/m-tmatma/sakura/build/1.0.645

参考
#313 (comment)

[beru]

AppVeyor のログを見て思ったのですが、

---- start python checkEncoding.py ----
('SHIFT_JIS', u'test.cpp')
return 1
error checkEncoding.py

ファイル名にリポジトリルートからのフォルダも出力されるかどうかが気になります。

この test.cpp ファイルが無いか https://github.com/m-tmatma/sakura/commits/feature/issue313-check-encoding を漁ってみましたが見つかりませんでした。

[m-tmatma]

この test.cpp ファイルが無いか https://github.com/m-tmatma/sakura/commits/feature/issue313-check-encoding を漁ってみましたが見つかりませんでした。

それを入れると、PR が失敗するので失敗することを確認した後、rebase して消しました。

[berryzplus]

これってどういうステータスですか？

試しに書いてみた or 入れる気でPR出した

変換対応が済んでないファイルがあるので、いまいれちゃマズい気がしています。

[m-tmatma]

これってどういうステータスですか？

マージしても問題ない認識です。

変換対応が済んでないファイルがあるので、いまいれちゃマズい気がしています。

cpp/h ファイルに関しては変換が完了している認識です。

バッチファイル等に関しては対象外です。

[m-tmatma]

なお、#313 (comment) のロジックを
使っていて、変更のあったファイルのみチェックすることで実行速度を稼いでいるのと
変更していないファイルをチェックしないようにすることで意図せず PR が失敗しないように
しています。

[beru]

処理的には動いてそうなので問題無いと思います。

検出時のメッセージにフォルダ部分が出るか分かりませんが、仮にもし出なかったとしたら出したい人が手直しすればすればよいですね。

[m-tmatma]

検出時のメッセージにフォルダ部分が出るか分かりませんが、仮にもし出なかったとしたら出したい人が手直しすればすればよいですね。

feature/issue313-check-encoding のブランチから分岐して、sakura_core\CGrepAgentSJIS.cpp を Shift-JIS で追加して、ビルドに失敗することを確認したものです。
https://ci.appveyor.com/project/m-tmatma/sakura/build/1.0.647/job/q35nnrh7hy5f6jqy

[beru]

feature/issue313-check-encoding のブランチから分岐して、sakura_core\CGrepAgentSJIS.cpp を Shift-JIS で追加して、ビルドに失敗することを確認したものです。
https://ci.appveyor.com/project/m-tmatma/sakura/build/1.0.647/job/q35nnrh7hy5f6jqy

フォルダパスもちゃんと出ていますね。確認ありがとうございます。

[m-tmatma]

https://github.com/sakura-editor/sakura/wiki に Pull request を送る際の文字コードに関して を追加しました。