sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.26k stars 163 forks source link

GHAのアクションを更新 #1908

Closed berryzplus closed 1 year ago

berryzplus commented 1 year ago

PR対象

カテゴリ

PR の背景

仕様・動作説明

GHAで使うactionsを最新版に変更します。

PR の影響範囲

GHAのビルド実行に影響する変更です。

テスト内容

CIビルドスクリプトの修正なのでテスト不可です。

v2をv3に変えるだけの修正なのでよほど何も起きないと思われます。

このPRのビルドでは変更結果が反映されないと思われます。

関連 issue, PR

参考資料

m-tmatma commented 1 year ago

改行コードの差分に関しては、編集した時の PC の core.autocrlf の設定によるのかなと思います。

Linux 上で、core.autocrlf が input でチェックアウトしたときはこの PR と master で 改行コードの違いを確認できました。(CRLF → LF) になってました。

一方、Windows で core.autocrlf が true の設定でチェックアウトしたときには、 変更前、変更後で両方とも CRLF になってました。 https://qiita.com/fronoske/items/a110a34bb05d76757e7d

編集環境によって、改行コードがころころ変わるとややこしいので .gitattributes で 規定してしまうというのがいいかと思います。 https://qiita.com/nacam403/items/23511637335fc221bba2

現状の CRLF を正とするか、git 標準の LF で正規化するかどちらも一理あるかと。

berryzplus commented 1 year ago

改行コードの変更は誤りなので修正します。

berryzplus commented 1 year ago

.github/workflows/build-sakura.ymlの文字コードをLFに戻してgit push -fしました。

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

berryzplus commented 1 year ago

レビューありがとうございます。マージしてしまいます。