Skip to content

sonar.cfamily.compile-commandsを使う#2096

Merged
berryzplus merged 2 commits into
sakura-editor:masterfrom
berryzplus:feature/use_sonar-cfamily-compile-commands
Jun 29, 2025
Merged

sonar.cfamily.compile-commandsを使う#2096
berryzplus merged 2 commits into
sakura-editor:masterfrom
berryzplus:feature/use_sonar-cfamily-compile-commands

Conversation

@berryzplus

@berryzplus berryzplus commented Jun 28, 2025

Copy link
Copy Markdown
Contributor

PR対象

  • ビルド手順/CI

カテゴリ

  • 改善

SonarQube側の最新仕様に合わせる。

PR の背景

sonarcloudの解析オプション sonar.cfamily.build-wrapper-output がdeprecatedになっています。

image

仕様・動作説明

sonarcloudの解析オプションを変更します。

sonar.cfamily.build-wrapper-output (deprecated)
 ↓
sonar.cfamily.compile-commands

PR の影響範囲

GitHub Actions の sonarscan に影響する変更です。
SonarQube cloudに表示される警告が消えます。

テスト内容

SonarQube cloudの警告が消えていること。

関連 issue, PR

参考資料

SonarQube側の更新に合わせる。
@berryzplus

Copy link
Copy Markdown
Contributor Author

SonarQube cloudの警告が消えていること。

確認。

image

一番下の「See analysis details on SonarQube Cloud」のところから飛んで、右上辺りの警告エリアに何も表示されてないことを確認。エビデンスは出しづらい。

@berryzplus berryzplus marked this pull request as ready for review June 28, 2025 05:44
@sonarqubecloud

Copy link
Copy Markdown

@m-tmatma

Copy link
Copy Markdown
Member

master では失敗している

https://sonarcloud.io/summary/new_code?id=sakura-editor_sakura&branch=master

image

この PR では成功している

https://sonarcloud.io/summary/new_code?id=sakura-editor_sakura&pullRequest=2096

image

こういうこと?

@m-tmatma

Copy link
Copy Markdown
Member

@berryzplus

berryzplus commented Jun 28, 2025

Copy link
Copy Markdown
Contributor Author

masterがfailedのキャプチャの右上に出てる has warning が このPRの分では消えてる で見てください。

masterがfailedでこのPRがSuccessになってるのは検証範囲の違いなので関係ありません。


参考リンクは
https://docs.sonarsource.com/sonarqube-server/10.8/analyzing-source-code/languages/c-family/running-the-analysis/
これ?

それで合ってます。参考資料を更新しました。

@berryzplus berryzplus merged commit 9d6c186 into sakura-editor:master Jun 29, 2025
15 checks passed
@berryzplus berryzplus deleted the feature/use_sonar-cfamily-compile-commands branch June 29, 2025 01:42
@beru beru added the SonarQube label Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants