? ? ? Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
13 Jul 2019

This updates PHPCS to version 3 and fixes all violations. This requires joomla/cms-coding-standards#5 to be merged.

avatar Hackwar Hackwar - open - 13 Jul 2019
avatar Hackwar Hackwar - change - 13 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jul 2019
Category Unit Tests Administration com_languages External Library Composer Change Libraries
avatar wilsonge
wilsonge - comment - 13 Jul 2019

I guess it goes without saying drone doesn't think this works :) Maybe the cache?

avatar Hackwar
Hackwar - comment - 13 Jul 2019

As I wrote above, the other PR to cms-coding-standards has to be merged first for this to work.

avatar wilsonge
wilsonge - comment - 13 Jul 2019

Tagged an alpha2 so please use that rather than just dev

avatar Hackwar Hackwar - change - 13 Jul 2019
Labels Added: ? ? ?
avatar Hackwar
Hackwar - comment - 13 Jul 2019

I have no idea what I have to put into the composer.json for that. Also, the tag says that there are 0 commits in that tag. Otherwise this would be ready for now.

avatar wilsonge
wilsonge - comment - 17 Jul 2019

Now the alpha has been released please use the tagged version rather than the branch

avatar wilsonge
wilsonge - comment - 17 Jul 2019

We seems to have significantly less assertions in PHPCS 3 than 2. Is there a reason for this?

avatar 810
810 - comment - 19 Jul 2019

v2 runs deprecated code filters also there where some exclude filter issues, now on v3 it has been resolved.

avatar rdeutz rdeutz - change - 19 Jul 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-19 15:26:19
Closed_By rdeutz
avatar rdeutz rdeutz - close - 19 Jul 2019
avatar rdeutz rdeutz - merge - 19 Jul 2019

Add a Comment

Login with GitHub to post a comment