-
Notifications
You must be signed in to change notification settings - Fork 46
PSR Support #77
Comments
Hoa already uses PSR-2 standards. Code is fixed by PHP-CS-Fixer by |
Other than this, there are other code problems which are not associated with PSR:
=\ |
This algorithm is particularily difficult to implement, and this is an optimisation. I dare you to build the same algorithm with random PSR conventions.
“should not”, this is not a “must not” :-). No violation.
For what I checked, there are full of jokes, stupid code duplication like this: It gives me an empty page, all sections are empty. I am totally willing to change Hoa's code, but a compiler is difficult to implement and PSR-2 are only about code style, that we respect :-). |
:D
=\
The link leads to the overall statistics of the code, the places that can be improved. Maybe permissions =( This is a free service (e.g. you can create the same inspection for hoa) that improves readability and code support. It sometimes buggy, but very often gives very useful advice (for example, about duplicating the code, as you can see). |
I think Scrutinizer can help having a better quality for sure. However, this issue is related to PSR. I guess we are OK with that. If you feel a boost of energy, you can open on issue on https://github.com/hoaproject/Central related to Scrutinizer. I am sure some @hoaproject/hoackers will take it :-). Thanks for the reports! |
What do you think about supporting generally accepted standards? e.g.:
Now the code of Hoa looks ... In Baroque style and reading it is quite problematic %)
The text was updated successfully, but these errors were encountered: