-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add CLI param to group output by rules #187
Comments
I suppose that choose just one of the two ways would be better.. |
By grouping by rules you can focus more quickly on resolving in your source code. Also, I would like to use this approach as the default approach, but I think also that it's best give to the developers the option to switch to the file-by-file approach based on their preferences or needs at a specific time. |
My 2cents: consistency with other tools in the PHP ecosystem is also important so I would check what other tools like psalm and deptrac do and stick to that |
Thanks @micheleorselli, I think your suggestion is very acceptable, but in my head the reasoning was this:
Also, I believe that in my very little experience with tools like psalm the approach is rightly the same of PHPUnit. |
Psalm, PHPUnit and deptrac I think that they don't allow you to group by rules instead of files. Have you ever tried those tools grouping by rules @ricfio ? |
@AlessandroMinoccheri In my opinion PHKArkitect should be more focused on rules (involving the architecture of the whole application), so could be more useful grouping by rules instead that by files... or simply to have this option. |
In my opinion, it's better to have the list of files sorted by name because I can fix those in order. |
Feature Request
Add CLI param to request output grouped by rules instead of by classes (actual default)
Summary
Such as above
The text was updated successfully, but these errors were encountered: