静的解析SonarCloudの対象を1環境に絞る

[berryzplus]

PR の目的

SonarCloud(SonarQube)の解析対象を1環境に絞ります。

カテゴリ

• ビルド関連
  • Azure Pipelines
  • GitHub Actions

※現状ではGitHub Actionsとは無関係ですが、GitHub Actionsでの解析を開始させるための前提として必要な修正と考えています。

PR の背景

#1476 で報告した問題の一部を解決します。

解析結果の過去断面を表示する方法が分からないので、現状では実行したビルド４つのうち３つが無駄になっています。

--	説明
現状	解析対象は4環境です。「用意ドン！」でスタートした4ビルドのうち、たまたま最後に終わった解析の結果だけを参照できます。
PR後	解析対象を1環境に固定します。解析した結果を無駄にすることなく表示することができるようになります。

PR のメリット

静的解析の対象ビルド環境を固定することにより、現状で発生している「ビルド環境により検出される警告の数が違う」という謎の現象の影響を受けにくくなります。

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

解析結果が4環境のどれになるか分からない、という軽くロシアンルーレットな気分を今後は味わえなくなります。

仕様・動作説明

仕様・動作の変更はありません。

PR の影響範囲

プログラムの挙動には影響を与えない変更です。
毎日 00:00JST に走る、artifactを生成しない静的解析専用のビルドに影響します。
通常のAzure Pipelinesビルドにも影響はありません。

テスト内容

SonarCloud上で sakura-editor/sakura のクローンを解析して、このPRの修正で対象が1環境になることを確認しました。
https://sonarcloud.io/dashboard?id=berryzplus_sakura
※検出されている警告が少ない原因は不明ですが、JDKのバージョンが関係している可能性があります。（別で対処します。）

関連 issue, PR

#510
#674
#1476

参考資料

berryzplus@c597ea7

[kengoide]

SonarCloud上で sakura-editor/sakura のクローンを解析して、このPRの修正で対象が1環境になることを確認しました。
https://sonarcloud.io/dashboard?id=berryzplus_sakura

（自分でも言ってたことですから）変更内容は理解できるのですが、このテスト内容を第三者が確認する方法が見出せずどうしていいかわからない状態です。

[berryzplus]

このテスト内容を第三者が確認する方法が見出せずどうしていいかわからない状態です。

1環境のみのビルドがスケジュールビルドされたのを確認できればOKかな～と思っています。
自分のとこのスケジュールビルドが壊れてて最近の自動解析が走っていなかったので直しておきました。

4:00に定期実行するようになってるので夜にでも確認をお願いします。

[kengoide]

問題ないものと思います。

[sanomari]

参考です。

変更前（このプロジェクトの現在の解析結果）

テスト内容のリンクから飛んだ先

キャプチャの見方

右側に出てる解析履歴が、変更前は同じのが2つ出ていて、変更後は1つになっています。
4つじゃないんですか！と思いましたが大きな問題でもないと思います。
どちらかというと検出されたバグ件数が全然違うことのほうが気になります。

私もこの変更で問題なさそうに思います。

[berryzplus]

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