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

checkstyle.xmlに汎用検査例外のキャッチを禁止する設定を追加 #1874

Merged

Conversation

kenjiyoshid-a
Copy link
Contributor

@kenjiyoshid-a kenjiyoshid-a commented Jan 20, 2025

この Pull request で実施したこと

Checkstyle の設定を行う checkstyle.xml に汎用検査例外のキャッチを禁止する IllegalCatch の設定を追加しました。
IllegalCatch タグはデフォルトで以下のキャッチを禁止します。

  • Error
  • Exception
  • RuntimeException
  • Throwable
  • java.lang.Error
  • java.lang.Exception
  • java.lang.RuntimeException
  • java.lang.Throwable

また、コーディング規約にCheckstyleプラグインのIllegalCatchを追加する旨を追記しました。

この Pull request では実施していないこと

  • デフォルトで設定されている例外以外のキャッチの禁止はしていません。

Issues や Discussions 、関連する Web サイトなどへのリンク

@kenjiyoshid-a kenjiyoshid-a linked an issue Jan 20, 2025 that may be closed by this pull request
@kenjiyoshid-a kenjiyoshid-a added this to the v1.1.0 milestone Jan 20, 2025
@kenjiyoshid-a kenjiyoshid-a added target: Dressca サンプルアプリケーションDresscaに関係がある target: Azure AD B2C Auth Azure AD B2C認証の要件別サンプル(コード)に関係がある labels Jan 20, 2025
@kenjiyoshid-a kenjiyoshid-a self-assigned this Jan 20, 2025
@kenjiyoshid-a kenjiyoshid-a added the target: ガイド/規約 ドキュメントのガイド/規約に関係がある label Jan 24, 2025
@tsuna-can-se tsuna-can-se requested a review from kharada7 January 27, 2025 09:43
@kharada7 kharada7 merged commit d486bd0 into main Jan 28, 2025
10 checks passed
@kharada7 kharada7 deleted the feature/checkstyle.xmlにIllegalCatchの設定を追加する branch January 28, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: Azure AD B2C Auth Azure AD B2C認証の要件別サンプル(コード)に関係がある target: Dressca サンプルアプリケーションDresscaに関係がある target: ガイド/規約 ドキュメントのガイド/規約に関係がある
Projects
None yet
Development

Successfully merging this pull request may close these issues.

checkstyle.xmlに<module name="IllegalCatch"/>を追加したい
2 participants