PSR Support about compiler HOT 7 CLOSED

Hoa already uses PSR-2 standards. Code is fixed by PHP-CS-Fixer by hoa devtools:cs (see https://github.com/hoaproject/Devtools/tree/master/Resource/PHPCSFixer for all configurations and extra fixers).

from compiler.

Hoa already uses PSR-2 standards.

@Hywan Are mutable objects (in particular, the transfer of arguments by reference, like this) are not side effects?

from compiler.

Besides:

Property names SHOULD NOT be prefixed with a single underscore to indicate protected or private visibility.

The fields declared in this way violate the PSR

from compiler.

Other than this, there are other code problems which are not associated with PSR:

1. https://scrutinizer-ci.com/g/SerafimArts/Compiler/?branch=master
2. https://scrutinizer-ci.com/g/SerafimArts/Compiler/code-structure/master

=\

from compiler.

Hoa already uses PSR-2 standards.
@Hywan Are mutable objects (in particular, the transfer of arguments by reference, like this) are not side effects?

This algorithm is particularily difficult to implement, and this is an optimisation. I dare you to build the same algorithm with random PSR conventions.

Property names SHOULD NOT be prefixed with a single underscore to indicate protected or private visibility.
The fields declared in this way violate the PSR

“should not”, this is not a “must not” :-). No violation.

Other than this, there are other code problems which are not associated with PSR:

1. https://scrutinizer-ci.com/g/SerafimArts/Compiler/?branch=master

For what I checked, there are full of jokes, stupid code duplication like this:

2. https://scrutinizer-ci.com/g/SerafimArts/Compiler/code-structure/master

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 :-).

from compiler.

This algorithm is particularily difficult to implement, and this is an optimisation. I dare you to build the same algorithm with random PSR conventions.

:D

“should not”, this is not a “must not” :-). No violation.

=\

It gives me an empty page, all sections are empty.

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).

from compiler.

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!

from compiler.