Skip to content

Rule to enforce class naming #230

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

Open
aivchen opened this issue May 16, 2025 · 2 comments
Open

Rule to enforce class naming #230

aivchen opened this issue May 16, 2025 · 2 comments

Comments

@aivchen
Copy link

aivchen commented May 16, 2025

Continuation of #214

What if we introduce optional path parameter to phpunit.xml in the same manner as containerXmlPath in phpstan/phpstan-symfony package? Then we can analyse suffix option for each directory and check all class names.

@ondrejmirtes
Copy link
Member

Isn't this something that https://github.com/shipmonk-rnd/dead-code-detector could take care of once it can detect unused classes? https://github.com/shipmonk-rnd/dead-code-detector

@aivchen
Copy link
Author

aivchen commented May 16, 2025

Well, maybe. We can look at this problem from different angles. We can think that some class is unused. But we also can think that some class has a wrong name.

Also we can interpret "unused class" in different ways. What if some test class with wrong name referenced by another class? Is it used or not then?

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

No branches or pull requests

2 participants