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

未使用関数GetExistPathと未使用の関数宣言を削除する #1517

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

魅しよう関数を削除することにより、SonarCloud解析のBugsに対処します。

カテゴリ

  • リファクタリング

PR の背景

SonarCloud解析でBugsが検出されてたコードをチェックしたところ、完全に未使用な関数が残っていることが分かったので該当箇所を削除します。

PR のメリット

SonarCloud解析で検出されているいくつかのBugs指摘に対処できます。

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

とくにないと思います。

仕様・動作説明

削除対象はどこからも参照されない未使用関数です。

PR の影響範囲

ありません。

テスト内容

削除だけなので不要と考えられます。

関連 issue, PR

#1504

参考資料

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

@kengoide
Copy link
Member

kengoide commented Jan 24, 2021

Approveしました。ビルドができている時点で問題を生じる余地はないはずです。

@berryzplus
Copy link
Contributor Author

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

@berryzplus berryzplus merged commit 408c273 into sakura-editor:master Jan 24, 2021
@berryzplus berryzplus deleted the feature/remove_unused_functions branch January 24, 2021 13:19
@beru beru added the refactoring リファクタリング 【ChangeLog除外】 label Mar 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring リファクタリング 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants