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

Error: * [The use of final classes is prohibited] #40

Merged
merged 1 commit into from
May 7, 2024

Conversation

ucan-lab
Copy link
Owner

@ucan-lab ucan-lab commented May 7, 2024

User description

close #39


PR Type

enhancement


Description

  • src/config/insights.phpForbiddenFinalClassesクラスを追加し、finalクラスの使用を禁止する設定を強化しました。

Changes walkthrough 📝

Relevant files
Enhancement
insights.php
PHP設定に`ForbiddenFinalClasses`クラスを追加                                           

src/config/insights.php

  • ForbiddenFinalClassesクラスを追加
+1/-0     

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

@github-actions github-actions bot added the bug 種別: 不具合の修正 label May 7, 2024
Copy link

github-actions bot commented May 7, 2024

PR Description updated to latest commit (9e492bd)

Copy link

github-actions bot commented May 7, 2024

PR Review 🔍

⏱️ Estimated effort to review [1-5]

1, このPRは非常に小規模で、変更点が明確であるため、レビューにかかる時間と労力は少ないです。

🧪 Relevant tests

No

⚡ Possible issues

No

🔒 Security concerns

No

Code feedback:

Copy link

github-actions bot commented May 7, 2024

PR Code Suggestions ✨

CategorySuggestions                                                                                                                                                       
Possible issue
最終クラスの使用禁止がプロジェクトに適切かどうかを確認してください。

ForbiddenFinalClasses::class
を追加することで、最終クラスの使用が禁止されますが、この変更が意図したものであるか確認してください。プロジェクトの要件によっては、最終クラスの使用が必要な場合もありますので、この変更が他の開発者に影響を与えないか検討することが重要です。

src/config/insights.php [82]

+// 確認後、問題なければこの行を保持
 ForbiddenFinalClasses::class,
 

@ucan-lab ucan-lab merged commit fb5076d into main May 7, 2024
6 checks passed
@ucan-lab ucan-lab deleted the fix-39-final-class branch May 7, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 種別: 不具合の修正
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error: * [The use of final classes is prohibited]
1 participant