-
Notifications
You must be signed in to change notification settings - Fork 261
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
First thoughts about a roadmap for the project #386
Comments
Hi @gmponos About V2 :
About V3 :
Thank you again! |
ofc.
are you suggesting to have only one config type (like only yaml)?
Thing is that although PSR-12 does not say it explicitly I believe it is more suited to projects that do not have active support of PHP5. For instance PSR-12 enforces visibility on constants. This feature does not exist on PHP 5. So eventually we will need to exclude some rules if we use PSR-12. That's why I mentioned PSR-2. Maybe for v3.0 we can think of applying PSR-12 instead.. Will add it in the description above.
Yes typo there.. fixed it above... |
I have also one question. Checking the v1 vs v2 I can see that some packages were removed and custom classes were created instead. May I ask why? I checked the issues #220 and #221 but I can not see the motivation behind it? What were the problems raised by using the symfony package? @Halleck45 maybe you could help here? |
Yes. I've removed plugins in order to make code more stable. Having plugins is a good idea and is useful for final user. Remiving them was not an objective of v2. Concerning symfony/console, I encoutered lot or bugs and dependencies issues (I whish support old versions of PHP : PhpMetrics is useful for old projects). Also with dependencies the phar archive was really big. Finally, I think that the Hoa/Ruler (or symfony/ewpression-language) must come back. The --condition-of-failure was very useful. Thanks for this issue ! 🙂 |
If you want to add support for phpcs, just let the developers to set the standard to use. |
@Mte90 I am not sure if we are discussing about the same thing.. I am not thinking about integrating codesniffer and phpstan inside the reports but I am thinking about checking the code of the repo against PhpStan and Codesniffer.
are you referring to my suggestion about dropping PHP5... yes I believe the same. Also I think it will take a while until we reach dropping support for PHP 5. There are many things to do until then.. until then let's see how this will go... :) |
uuuhhh got it, my fault. |
Never the less if we ever make it possible for the developers to register their own extensions/modules then modules like phpstan or codesniffer could be interesting... |
These are my first thoughts about the package
Version 2
Version 3
PHP 7PHP 5. My initial thought is require 7.2 as min.Any feedback or help is welcome.
The text was updated successfully, but these errors were encountered: