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.

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

SonarCloud静的解析でスタブDLLのソースを除外する #1816

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

SonarCloud静的解析の設定を変更します。

カテゴリ

  • ビルド関連
    • GitHub Actions

PR の背景

#1815 で外部DLLの動的アクセスクラスの自動テスト導入を検討した際に、
テスト用に追加するスタブDLLのソースコードが静的解析されると不都合だと分かりました。

テスト専用のコードだからって気を抜かずにちゃんと書けよ!
ってツッコミはあると思うんですが、
テストコード(test-*.cpp)は現状で既に除外対象になってるので、
レベルを合わせるためにもスタブDLLのソースコードを除外対象としたいです。

同時に、除外設定の見直しを行って設定を健全化したいと思います。

PR のメリット

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

仕様・動作説明

SonarCloud静的解析の設定を変更します。
アプリの仕様・機能には影響しません。

変更点

  • 使われなくなった中間フォルダの除外指定を外します。
  • スタブDLLのソースがおかれるフォルダを静的解析の除外対象にします。
  • スタブDLLのソースファイルをカバレッジ測定の対象から外します。
  • Pythonスクリプトの静的解析をver3想定で行うようにします。
    「ver未指定だとちゃんとした警告が出せない」というワーニングが出ていました。
    Python ver2系のサポートは終了している認識なので、ver3ということにします。

PR の影響範囲

SonarCloud静的解析の結果に影響します。

テスト内容

変更結果はマージ後に反映される仕組みなため、事前にテストすることはできません。

関連 issue, PR

参考資料

作らなくなったので削除
スタブDLLのソースコードを静的解析対象から外す。
@sonarcloud
Copy link

sonarcloud bot commented Mar 7, 2022

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
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。
何か不都合あれば別PRで対応していきたいと思います。

@berryzplus berryzplus merged commit 4b1d1f0 into sakura-editor:master Mar 7, 2022
@berryzplus berryzplus deleted the feature/maintenance_sonarscan branch March 7, 2022 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants