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

Consider Droping PHPCS 2.x support by August 31, 2018 #227

Open
photodude opened this issue Feb 9, 2018 · 6 comments
Open

Consider Droping PHPCS 2.x support by August 31, 2018 #227

photodude opened this issue Feb 9, 2018 · 6 comments

Comments

@photodude
Copy link
Contributor

Following what other Coding standards using PHPCS are doing, We should consider dropping PHPCS 2.x support by August 31, 2018

Example of what another Coding standard is doing (details of why are dropping support in August are included in their list)
WordPress/WordPress-Coding-Standards#1048

@photodude
Copy link
Contributor Author

Note: The next release of PHPCS 2.9.x will be the last bug fix for that branch.

@wilsonge
Copy link
Contributor

wilsonge commented Feb 9, 2018

I think we'll leave the J3.x series on PHPCS 2.x series and J4.x will go straight onto PHPCS 3.x

@photodude
Copy link
Contributor Author

photodude commented Feb 9, 2018

Framework 2.x should also move onto PHPCS 3.x
Kinda always been what I suggested since the creation of the PHPCS 3.x branch. If you don't support PHP 5.3 you should plan on using PHPCS 3.x

Of course, this is just a tool that we typically run on PHP 5.6+ so do we even care about the lack of PHP 5.3 support with the tool?

In any case, we need to get a PHPCS2.x stable releases and a PHPCS 3.x release out.

@mbabker
Copy link
Contributor

mbabker commented Feb 9, 2018

Framework 2.x should also move onto PHPCS 3.x

We'll get there. I'm still working through dropping the gitmodule for 1.x and using Composer for 2.x across repos. The ones that are already done is as easy as changing a version constraint, the rest still need some TLC.

@mbabker
Copy link
Contributor

mbabker commented Feb 9, 2018

Of course, this is just a tool that we typically run on PHP 5.6+ so do we even care about the lack of PHP 5.3 support with the tool?

Yes. If someone clones a PHP 5.3 compat repo on a PHP 5.3 system they should be able to run all related test tooling and not rely on our CI systems to run tests for them. It's part of the reason we're still going through the pain of keeping PHPUnit compatible with 4.8 even though there are now 3 newer major series.

@jrfnl
Copy link
Contributor

jrfnl commented Mar 5, 2018

Hiya, WordPress Coding Standards contributor here!

Just FYI as I noticed the WPCS ticket is referenced here:

WPCS has had support for PHPCS 2.x as well as 3.x since August 2017, so we currently support both versions, though we encourage people to use PHPCS 3.x.

We did this as there are a number of company/other community standards within the WP community which depend on WPCS.
This way, those other standards will have a years time to upgrade their standard to be compatible with PHPCS 3.x
So when we drop support for PHPCS 2.x by end of August 2018, it will be after a year of cross-version PHPCS version support.

As a side-note - the PHPCompatibility standard is compatible with PHPCS 1.5.6, 2.2+ and 3.0.2+ and intends to maintain PHPCS cross-version support.

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

4 participants