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

Cleanup codebase #105

Open
lchrusciel opened this issue Dec 20, 2017 · 6 comments
Open

Cleanup codebase #105

lchrusciel opened this issue Dec 20, 2017 · 6 comments

Comments

@lchrusciel
Copy link
Owner

We should try some tool that will help to maintain clean code base.

E.g. https://github.com/Symplify/EasyCodingStandard

@lchrusciel lchrusciel added this to the v.3.0.0 milestone Dec 20, 2017
@lchrusciel lchrusciel removed this from the v.3.0.0 milestone Mar 6, 2018
@antkaz
Copy link

antkaz commented Nov 6, 2018

I can advise https://github.com/slevomat/coding-standard

@lchrusciel
Copy link
Owner Author

Isn't it part of EsayCodingStandard?

Anyway, I have started to work on ECS integration in #137, but maybe I will give a try to Slevomat. Also, feel free to help me :)

@antkaz
Copy link

antkaz commented Nov 6, 2018

Not quite.
I am not familiar with EasyCodingStandard, but judging by the documentation, it uses the rules of the PHP_CodeSniffer.

Levomat also uses code-sniffer, but it also contains many additional rules (they also exist in phpstan) that can be customized for your project.

In my project, I started using PHP_CodeSniffer, but then I added the Slevomat rules and configured them as necessary (which does not allow PHP_CodeSniffer to do).

@lchrusciel
Copy link
Owner Author

I truly believe that ECS is just a wrapper around Slevomat, as there are a lot of Slevomat rules used internally like here: https://github.com/Symplify/EasyCodingStandard/blob/master/config/clean-code.yml#L2-L12

Still, anything that will make the code better by automatization is good for me :)

@stefandoorn
Copy link

What about the Sylius Coding Standard rules for PHPStan?

@antkaz
Copy link

antkaz commented Nov 6, 2018

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

3 participants